---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/39525/#review103477
---
Master (5609fc2) is red with this patch.
./build-support/jenkins/build.sh
:processJmhResources UP-TO-DATE
:jmhClasses
:checkstyleJmh
:jsHint
:checkstyleMain
:compileTestJavaNote: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
:processTestResources
:testClasses
:checkstyleTest
:findbugsJmh
:findbugsMain
:findbugsTest
:licenseJmh UP-TO-DATE
:licenseMain UP-TO-DATE
:licenseTest UP-TO-DATE
:license UP-TO-DATE
:pmdMain
:test
:jacocoTestReport
Coverage report generated:
file:///home/jenkins/jenkins-slave/workspace/AuroraBot/dist/reports/jacoco/test/html/index.html
:analyzeReport
Instruction coverage is 0.8873498583862232, but must be greater than 0.89
Branch coverage is 0.8132474701011959, but must be greater than 0.835
:analyzeReport FAILED
FAILURE: Build failed with an exception.
* What went wrong:
Execution failed for task ':analyzeReport'.
> Test coverage missing for
> org/apache/aurora/scheduler/http/api/GsonMessageBodyHandler$2
* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug
option to get more log output.
BUILD FAILED
Total time: 5 mins 17.366 secs
I will refresh this build result if you post a review containing "@ReviewBot
retry"
- Aurora ReviewBot
On Oct. 21, 2015, 8:56 p.m., Kevin Sweeney wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/39525/
> ---
>
> (Updated Oct. 21, 2015, 8:56 p.m.)
>
>
> Review request for Aurora, Maxim Khutornenko and Zameer Manji.
>
>
> Repository: aurora
>
>
> Description
> ---
>
> This change ignores all SessionKeys. Since the scheduler was doing
> "requireNonNull" on SessionKey before the client needs to continue to set
> SessionKey to a non-null value for this release. In the next release the
> parameter can be dropped completely.
>
> Apologies for the large diff, thankfully it's mostly red. Coverage in
> SchedulerThriftInterface is very high, most of the deleted tests deal with
> branches related to authentication or authorization failures, as these are
> now enforced in a different layer.
>
>
> Diffs
> -
>
> NEWS 7622f9db706c1764876f4174855343414dc59366
> src/main/java/org/apache/aurora/auth/CapabilityValidator.java
> 198cdf3a63195c6b125d6a2e5d726075fd41f809
> src/main/java/org/apache/aurora/auth/SessionValidator.java
> b688a0f0f1a4549dc4fe81f5cc50af1167856e87
> src/main/java/org/apache/aurora/auth/UnsecureAuthModule.java
> 476dd9a89ce317e3bd242fd70a55cc919aff8418
> src/main/java/org/apache/aurora/auth/UnsecureSessionContext.java
> 47f81aa4851c753779918dfeb137041073127cd4
> src/main/java/org/apache/aurora/scheduler/app/MoreModules.java
> b79ccc849834c9a7f7ac654e4302f2ba4e13ff66
> src/main/java/org/apache/aurora/scheduler/http/JettyServerModule.java
> ee024a34f1fdde49791ad6573fe5c0ae3953a7c7
> src/main/java/org/apache/aurora/scheduler/thrift/AuditMessages.java
> 3a29a628de8f1c1c3ed609b494656ac95db9efcf
>
> src/main/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterface.java
> 304437e95dd1692b65bf5d0a6697ab62127e6833
> src/main/java/org/apache/aurora/scheduler/thrift/aop/AopModule.java
> e951f8e55353ff3c8d7d0e86016a2a004137c299
>
> src/main/java/org/apache/aurora/scheduler/thrift/aop/LoggingInterceptor.java
> d110b21d4193eb37a10ee8ba2eca82d208496c38
>
> src/main/java/org/apache/aurora/scheduler/thrift/aop/UserCapabilityInterceptor.java
> dd8d272751034ef7daa720e12dd4ed850532da39
> src/main/java/org/apache/aurora/scheduler/thrift/auth/Requires.java
> 9b3431899cddfc7f949fc8cb12f6e4739b4989d0
> src/main/java/org/apache/aurora/scheduler/thrift/auth/ThriftAuthModule.java
> e1a21f632f2bee5834cbbde9d6902d71b6a090d9
> src/test/java/org/apache/aurora/auth/UnsecureSessionContextTest.java
> c3c34656d781bc54282206caf1211d1385fd68c8
> src/test/java/org/apache/aurora/scheduler/http/api/ApiBetaTest.java
> 8af7c46e46d2269903118deb2a2a2b61bcc1d8d1
> src/test/java/org/apache/aurora/scheduler/thrift/AuditMessagesTest.java
> 0f8748d2981d48236f602778d2c1848b4c572445
>
> src/test/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterfaceTest.java
> f537f7a39496f69953e6d86cb15359fe3a705480
> src/test/java/org/apache/aurora/scheduler/thrift/ThriftIT.java
> f06481bd237db0ab84850fe987d6e1fcb194f31b
> src/test/java/org/apache/aurora/scheduler/thrift/aop/AopModuleTest.java
> b940e82644365fae613d577ba5b8779077f115d4
>
> src/test/java/org/apache/aurora/scheduler/thrift/aop/LoggingInterceptorTest.java
> 1bb3eae224de287d0ed83534f1c933ddd72298bb
>
> Diff: