Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Develop #89

Merged
merged 20 commits into from
May 6, 2019
Merged

Develop #89

merged 20 commits into from
May 6, 2019

Conversation

jirenius
Copy link
Collaborator

@jirenius jirenius commented May 6, 2019

No description provided.

jirenius added 20 commits April 1, 2019 13:57
@jirenius
GH-76: Added addr configuration option.
004d377
@jirenius
Merge pull request #77 from jirenius/feature/gh-76-bind-to-host-address
40ec308 
GH-76: Added addr configuration option.
@jirenius
GH-78: Added failing test for the bug.
dfe4a5d
@jirenius
GH-78: Improved cyclic test code to cover more cases.
b084007
@jirenius
GH-78: Exchanging path slice with a refMap.
39bce3c
@jirenius
GH-78: Refactored OnLoaded structure into OnReady and separate onLoad…
5d70509 
…ed for internal use.
@jirenius
GH-78: Updated megacheck to staticcheck. Fixed errors found by static…
0dd6d7d 
…check.
@jirenius
Merge pull request #80 from jirenius/feature/gh-78-unable-to-resolve-…
73ea075 
…complex-recursion

Feature/gh 78 unable to resolve complex recursion
@jirenius
GH-84: Added test and fix for multiple response bug.
4404380
@jirenius
Merge pull request #85 from jirenius/bugfix/gh-84-multiple-responses-…
c7590f0 
…to-get-request

GH-84: Added test and fix for multiple response bug.
@jirenius
GH-86: Fixed crash bug on load error on disposed resources.
5c5b82e 
Added cyclic HTTP get tests.
@jirenius
Merge pull request #87 from jirenius/bugfix/gh-86-crash-on-timed-out-…
882c623 
…resources

GH-86: Fixed crash bug on load error on disposed resources.
@jirenius
GH-82: Added support for api encoding: jsonFlat
8244c67
@jirenius
GH-82: Implemented API encoder for JSON, and the use of APIEncoderFac…
4010675 
…tory. Added tests for encoders.
@jirenius
GH-82: Structured test with sequential event tables. Fixed race bug i…
c70d497 
…n apiEncoding.
@jirenius
GH-82: Improved invalid config error message.
15f0f74
@jirenius
GH-82: Fixed test to handle NewService error.
4760a35
@jirenius
GH-82: Updated README with apiEncoding config. Fixed config tests.
543d0e9
@jirenius
Merge pull request #88 from jirenius/feature/gh-82-api-encoding-confi…
5cfc88f 
…guration

Feature/gh 82 api encoding configuration
@jirenius
GH-82: Removed redundant httpapi.
a1a67e9
@jirenius jirenius merged commit 8b98e31 into master May 6, 2019
@coveralls
Copy link

Pull Request Test Coverage Report for Build 107

  • 412 of 495 (83.23%) changed or added relevant lines in 9 files are covered.
  • 1 unchanged line in 1 file lost coverage.
  • Overall coverage decreased (-0.004%) to 82.173%
Changes Missing Coverage Covered Lines Changed/Added Lines %
server/httpServer.go 4 6 66.67%
server/service.go 16 19 84.21%
server/wsConn.go 24 27 88.89%
server/config.go 21 29 72.41%
server/subscription.go 79 89 88.76%
server/apiHandler.go 35 48 72.92%
server/apiEncoding.go 229 273 83.88%
Files with Coverage Reduction New Missed Lines %
server/config.go 1 63.49%
Totals Coverage Status
Change from base Build 81: -0.004%
Covered Lines: 2443
Relevant Lines: 2973
💛 - Coveralls

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jirenius @coveralls