Skip to content

ServerProtocol with UnitTests #1

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jul 22, 2015
Merged

ServerProtocol with UnitTests #1

merged 1 commit into from
Jul 22, 2015

Conversation

benjchristensen
Copy link
Contributor

This is exploratory and still using ASCII rather than binary.

This is exploratory and still using ASCII rather than binary.
@benjchristensen
Copy link
Contributor Author

I don't understand why this is failing. It's like it's having issues with Java 8 types.

Here is my machine:

lgml-bechristensen:reactivesocket-java benjchristensen$ ./gradlew build
Inferred version: 0.0.1-SNAPSHOT
[buildinfo] Properties file path was not found! (Relevant only for builds running on a CI Server)
:compileJava
:processResources UP-TO-DATE
:classes
:writeManifestProperties
:jar
:javadoc
/Users/benjchristensen/development/github/reactivesocket-java/src/main/java/io/reactivesocket/Message.java:74: warning - @return tag has no arguments.
/Users/benjchristensen/development/github/reactivesocket-java/src/main/java/io/reactivesocket/Message.java:102: warning - @return tag has no arguments.
2 warnings
:javadocJar
:compilePerfJava UP-TO-DATE
:processPerfResources UP-TO-DATE
:perfClasses UP-TO-DATE
:shadowJar
:sourceJar
:assemble
:writeLicenseHeader
:licenseExamples UP-TO-DATE
:licenseMain UP-TO-DATE
:licensePerf UP-TO-DATE
:licenseTest
Missing header in: src/test/java/io/reactivesocket/TestConnection.java
:license
:compileTestJava
:processTestResources UP-TO-DATE
:testClasses
:test

io.reactivesocket.MessageTest > testWrapMessage STARTED

io.reactivesocket.MessageTest > testWrapBytes STARTED

io.reactivesocket.MessageTest > testWriteThenRead STARTED

io.reactivesocket.ReactiveSocketServerProtocolTest > testRequestResponseSuccess STARTED

io.reactivesocket.ReactiveSocketServerProtocolTest > testRequestResponseError STARTED

io.reactivesocket.ReactiveSocketServerProtocolTest > testRequestResponseCancel STARTED

io.reactivesocket.ReactiveSocketServerProtocolTest > testRequestStreamSuccess STARTED

io.reactivesocket.ReactiveSocketServerProtocolTest > testRequestStreamError STARTED

io.reactivesocket.ReactiveSocketServerProtocolTest > testRequestStreamCancel STARTED

io.reactivesocket.ReactiveSocketServerProtocolTest > testMultiplexedStreams STARTED
:check
:compileExamplesJava UP-TO-DATE
:processExamplesResources UP-TO-DATE
:examplesClasses UP-TO-DATE
:build

BUILD SUCCESSFUL

Total time: 7.916 secs
lgml-bechristensen:reactivesocket-java benjchristensen$ java -version
java version "1.8.0_45"
Java(TM) SE Runtime Environment (build 1.8.0_45-b14)
Java HotSpot(TM) 64-Bit Server VM (build 25.45-b02, mixed mode)
lgml-bechristensen:reactivesocket-java benjchristensen$ ./gradlew --version

------------------------------------------------------------
Gradle 2.2.1
------------------------------------------------------------

Build time:   2014-11-24 09:45:35 UTC
Build number: none
Revision:     6fcb59c06f43a4e6b1bcb401f7686a8601a1fb4a

Groovy:       2.3.6
Ant:          Apache Ant(TM) version 1.9.3 compiled on December 23 2013
JVM:          1.8.0_45 (Oracle Corporation 25.45-b02)
OS:           Mac OS X 10.10.4 x86_64

@benjchristensen
Copy link
Contributor Author

Merging anyways to keep progressing.

benjchristensen added a commit that referenced this pull request Jul 22, 2015
@benjchristensen benjchristensen merged commit 626c947 into master Jul 22, 2015
@benjchristensen benjchristensen deleted the server-protocol branch July 22, 2015 02:22
@benjchristensen
Copy link
Contributor Author

It was a JDK bug and TravisCI was using an old version. Fixed using tip from here: travis-ci/travis-ci#4042

See https://github.com/ReactiveSocket/reactivesocket-java/blob/eede001a12e15a974792ae7ea592b496fa6b4550/.travis.yml#L5

robertroeser added a commit that referenced this pull request Aug 17, 2015
bug fix and NEXT_COMPLETE support.
stevegury pushed a commit that referenced this pull request Jun 8, 2016
stevegury pushed a commit that referenced this pull request Jun 8, 2016
stevegury pushed a commit that referenced this pull request Jun 8, 2016
updated to use use reactivesocket 0.0.5, and added ability to schedul…
OlegDokuka added a commit that referenced this pull request Jun 5, 2020
Signed-off-by: Oleh Dokuka <[email protected]>
OlegDokuka added a commit that referenced this pull request Jun 17, 2020
Signed-off-by: Oleh Dokuka <[email protected]>
OlegDokuka added a commit that referenced this pull request Oct 13, 2020
Signed-off-by: Oleh Dokuka <[email protected]>
OlegDokuka added a commit that referenced this pull request May 19, 2021
Signed-off-by: Oleh Dokuka <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant