yarpc / yarpc-java

44%

master: 83%

LAST BUILD ON BRANCH findbugs-exclude

branch: findbugs-exclude

CHANGE BRANCH

x

Reset

• findbugs-exclude
• abshost
• add-deadline
• add-gradle-tasks
• add-python-sync
• build
• cache-thrift
• call-reply
• channel
• checkstyle
• cleangradle
• cleanup-gradle
• client
• close-oneway
• common-test-deps
• completion-stages-tests
• completionstage
• connection-reuse
• context
• context-timeout
• crossdock-echo-behaviors
• crossdock-errors
• crossdock-headers
• crossdock-headers-outbound
• crossdock-headers-outbound-json
• crossdock-headers-outbound-thrift
• crossdock-http-errors
• crossdock-library
• crossdock-python
• crossdock-python-async
• crossdock-raw-client
• crossdock-thrift-outbound
• crossdock-timeout
• crossdock-tracing
• crossdockrenaming
• data-model-update
• data-model-update-raw-encoding
• delete-tchannel
• dependencies
• dependency-management
• dev-version
• disable-crossdock
• disable-temporarily
• dispatcher-builder-outbound
• dispatcher-lifecycle
• dispatcher-register-varargs
• enable-crossdock
• enable-crossdock-logging
• enable-crossdock-travis
• enable-gauntlet
• enable-go-client
• enable-go-server
• enable-json-behavior
• enable-node-python-crossdock
• enable-oneway
• enable-oneway-outbound
• enable-oneway-outbound-json
• enable-oneway-outbound-thrift
• enable-oneway2
• enable-thrift-behavior
• enable-thrift-crossdock
• encoding-javadocs
• exception-handling
• fill-context
• findbugs
• fix-code-formatter
• fix-context-headers
• fix-crossdock-build
• fix-event-loop-group
• fix-exceptions
• fix-http
• fix-http-conversion
• fix-jacoco-coverage
• fix-json-ack
• fix-json-body
• fix-oneway
• fix-oneway-ack
• fix-test-coverage
• fix-test-timeout
• fix-thrift-procedure-name
• fix-thrift-type-conversion
• fix-travis-crossdock-logging
• fix-unary-oneway
• formatting
• generated-handler
• google-java-format
• gradle
• gradle-all
• gradle-clean
• gradle-exported-projects
• grpc-support
• grpc-support-2
• grpc-support-3
• grpc-support-4
• grpc-support-5
• handler-spec
• handlerwrapper
• header-crossdock
• http-bytes-copy
• http-close
• http-package
• http-transport
• http-transport-shutdown
• immutable-context
• inbound-set-router
• inject-tprotocolfactory
• integration
• integration-tests
• java8-only
• javadoc
• json-encoding
• license
• licenses
• lombok
• map-null
• master
• middleware
• minor-cleanup
• mockito-rule
• naming
• netty4.1
• new-crossdock
• no-joda
• oh-the-humanity
• oneway-json-crossdock
• procedure-factories
• procedure-service-name
• process-middleware
• properties
• public
• raw-client-test
• reduce-unchecked-cast
• registry
• release
• release-0.2.0
• remove-transportexception
• request-encoding-exception
• require-encoding
• response-encoding-exception
• response-headers
• rm-buffer-util
• rm-unused-exceptions
• rx
• rx-json-thrift
• server
• skip-unimplemented-behaviors
• smaller-docker-compose
• standard-checkstyle
• test
• threadlocal
• thrift-crossdock
• thrift-encoding
• thrift-fqcn
• thrift-generator
• thrift-integration-tests
• timeout-exception
• timeoutexception-message
• tracing
• transport-body2
• transport-package
• transportHandler
• travis
• travis-docker-env
• travis-fix
• travisbranchinenv
• travisfix
• unwrap-completionexception
• upgrade-docker
• upgrade-gjf
• upgrade-gradle
• upgrade-mockito
• use-rpc-prefix
• wait-for-python-sync
• yarpc-0.2.0
• yarpc-core
• yarpc-encoding-json
• yarpc-encoding-thrift
• yarpc-raw
• yarpcgodev
• yarpcgomaster

Build # 945

Build Type

Pull #214

travis-ci

Committed by web-flow

Commit Message

tests: Use MockitoRule with strict stubs

This replaces all uses of `MockitoAnnotations.initMocks` with
`MockitoRule` and [`STRICT_STUBS`].

MockitoRule automatically injects mocks for `@Mock` annotations and
verifies them to catch stubbing issues. This even affects mocks created
manually in that test with `Mockito.mock(..)`, hence the change in
DispatcherTest which doesn't have any `@Mock` annotations.

Note that strict stubs are  intended to be the default behavior in
Mockito 3. Until then we'll have to remember to pass the argument to the
rule everywhere we use mocks.

  [`STRICT_STUBS`]: http://static.javadoc.io/org.mockito/mockito-core/2.7.5/org/mockito/quality/Strictness.html#STRICT_STUBS

Pull Request Pull Request #214: tests: Use MockitoRule with strict stubs

Run Details

350 of 800 relevant lines covered (43.75%)

0.44 hits per line