yarpc / yarpc-java

44%

master: 83%

LAST BUILD ON BRANCH mockito-rule

branch: mockito-rule

CHANGE BRANCH

x

Reset

• mockito-rule
• 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
• findbugs-exclude
• 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
• 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 # 947

Build Type

push

travis-ci

Committed by abhinav

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

Run Details

350 of 800 relevant lines covered (43.75%)

0.44 hits per line