Skip to content

Fix app roles; introduce resource-specific role enums

Codecov / codecov/patch succeeded Sep 12, 2024 in 1s

82.11% of diff hit (target 74.36%)

View this Pull Request on Codecov

82.11% of diff hit (target 74.36%)

Annotations

Check warning on line 268 in core/src/main/scala/org/broadinstitute/dsde/workbench/leonardo/samModels.scala

See this annotation in the file changed.

@codecov codecov / codecov/patch

core/src/main/scala/org/broadinstitute/dsde/workbench/leonardo/samModels.scala#L268

Added line #L268 was not covered by tests

Check warning on line 272 in core/src/main/scala/org/broadinstitute/dsde/workbench/leonardo/samModels.scala

See this annotation in the file changed.

@codecov codecov / codecov/patch

core/src/main/scala/org/broadinstitute/dsde/workbench/leonardo/samModels.scala#L272

Added line #L272 was not covered by tests

Check warning on line 276 in core/src/main/scala/org/broadinstitute/dsde/workbench/leonardo/samModels.scala

See this annotation in the file changed.

@codecov codecov / codecov/patch

core/src/main/scala/org/broadinstitute/dsde/workbench/leonardo/samModels.scala#L276

Added line #L276 was not covered by tests

Check warning on line 282 in core/src/main/scala/org/broadinstitute/dsde/workbench/leonardo/samModels.scala

See this annotation in the file changed.

@codecov codecov / codecov/patch

core/src/main/scala/org/broadinstitute/dsde/workbench/leonardo/samModels.scala#L282

Added line #L282 was not covered by tests

Check warning on line 286 in core/src/main/scala/org/broadinstitute/dsde/workbench/leonardo/samModels.scala

See this annotation in the file changed.

@codecov codecov / codecov/patch

core/src/main/scala/org/broadinstitute/dsde/workbench/leonardo/samModels.scala#L286

Added line #L286 was not covered by tests

Check warning on line 290 in core/src/main/scala/org/broadinstitute/dsde/workbench/leonardo/samModels.scala

See this annotation in the file changed.

@codecov codecov / codecov/patch

core/src/main/scala/org/broadinstitute/dsde/workbench/leonardo/samModels.scala#L290

Added line #L290 was not covered by tests

Check warning on line 296 in core/src/main/scala/org/broadinstitute/dsde/workbench/leonardo/samModels.scala

See this annotation in the file changed.

@codecov codecov / codecov/patch

core/src/main/scala/org/broadinstitute/dsde/workbench/leonardo/samModels.scala#L296

Added line #L296 was not covered by tests

Check warning on line 301 in core/src/main/scala/org/broadinstitute/dsde/workbench/leonardo/samModels.scala

See this annotation in the file changed.

@codecov codecov / codecov/patch

core/src/main/scala/org/broadinstitute/dsde/workbench/leonardo/samModels.scala#L301

Added line #L301 was not covered by tests

Check warning on line 305 in core/src/main/scala/org/broadinstitute/dsde/workbench/leonardo/samModels.scala

See this annotation in the file changed.

@codecov codecov / codecov/patch

core/src/main/scala/org/broadinstitute/dsde/workbench/leonardo/samModels.scala#L305

Added line #L305 was not covered by tests

Check warning on line 311 in core/src/main/scala/org/broadinstitute/dsde/workbench/leonardo/samModels.scala

See this annotation in the file changed.

@codecov codecov / codecov/patch

core/src/main/scala/org/broadinstitute/dsde/workbench/leonardo/samModels.scala#L311

Added line #L311 was not covered by tests

Check warning on line 316 in core/src/main/scala/org/broadinstitute/dsde/workbench/leonardo/samModels.scala

See this annotation in the file changed.

@codecov codecov / codecov/patch

core/src/main/scala/org/broadinstitute/dsde/workbench/leonardo/samModels.scala#L316

Added line #L316 was not covered by tests

Check warning on line 320 in core/src/main/scala/org/broadinstitute/dsde/workbench/leonardo/samModels.scala

See this annotation in the file changed.

@codecov codecov / codecov/patch

core/src/main/scala/org/broadinstitute/dsde/workbench/leonardo/samModels.scala#L320

Added line #L320 was not covered by tests

Check warning on line 486 in http/src/main/scala/org/broadinstitute/dsde/workbench/leonardo/dao/HttpSamDAO.scala

See this annotation in the file changed.

@codecov codecov / codecov/patch

http/src/main/scala/org/broadinstitute/dsde/workbench/leonardo/dao/HttpSamDAO.scala#L486

Added line #L486 was not covered by tests

Check warning on line 1699 in http/src/main/scala/org/broadinstitute/dsde/workbench/leonardo/http/service/LeoAppServiceInterp.scala

See this annotation in the file changed.

@codecov codecov / codecov/patch

http/src/main/scala/org/broadinstitute/dsde/workbench/leonardo/http/service/LeoAppServiceInterp.scala#L1699

Added line #L1699 was not covered by tests

Check warning on line 342 in http/src/main/scala/org/broadinstitute/dsde/workbench/leonardo/http/service/RuntimeServiceInterp.scala

See this annotation in the file changed.

@codecov codecov / codecov/patch

http/src/main/scala/org/broadinstitute/dsde/workbench/leonardo/http/service/RuntimeServiceInterp.scala#L342

Added line #L342 was not covered by tests

Check warning on line 410 in http/src/main/scala/org/broadinstitute/dsde/workbench/leonardo/http/service/RuntimeServiceInterp.scala

See this annotation in the file changed.

@codecov codecov / codecov/patch

http/src/main/scala/org/broadinstitute/dsde/workbench/leonardo/http/service/RuntimeServiceInterp.scala#L410

Added line #L410 was not covered by tests

Check warning on line 1323 in http/src/main/scala/org/broadinstitute/dsde/workbench/leonardo/http/service/RuntimeServiceInterp.scala

See this annotation in the file changed.

@codecov codecov / codecov/patch

http/src/main/scala/org/broadinstitute/dsde/workbench/leonardo/http/service/RuntimeServiceInterp.scala#L1323

Added line #L1323 was not covered by tests

Check warning on line 57 in http/src/main/scala/org/broadinstitute/dsde/workbench/leonardo/monitor/AutoDeleteAppMonitor.scala

See this annotation in the file changed.

@codecov codecov / codecov/patch

http/src/main/scala/org/broadinstitute/dsde/workbench/leonardo/monitor/AutoDeleteAppMonitor.scala#L57

Added line #L57 was not covered by tests

Check warning on line 61 in http/src/main/scala/org/broadinstitute/dsde/workbench/leonardo/monitor/AutoDeleteAppMonitor.scala

See this annotation in the file changed.

@codecov codecov / codecov/patch

http/src/main/scala/org/broadinstitute/dsde/workbench/leonardo/monitor/AutoDeleteAppMonitor.scala#L60-L61

Added lines #L60 - L61 were not covered by tests

Check warning on line 204 in http/src/main/scala/org/broadinstitute/dsde/workbench/leonardo/monitor/NonLeoMessageSubscriber.scala

See this annotation in the file changed.

@codecov codecov / codecov/patch

http/src/main/scala/org/broadinstitute/dsde/workbench/leonardo/monitor/NonLeoMessageSubscriber.scala#L203-L204

Added lines #L203 - L204 were not covered by tests