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

chore: reorganise Package Structure #97

Merged
merged 67 commits into from
Sep 12, 2024
Merged

chore: reorganise Package Structure #97

merged 67 commits into from
Sep 12, 2024

Conversation

turtleDev
Copy link
Contributor

No description provided.

@turtleDev
publisher: log: add tests
dc1093c
@turtleDev
worker: test for ack function
a0203e1
@turtleDev
worker: tests: remove unused mocks
bad90fd
@turtleDev
worker: add tests for error cases
fb55724
@turtleDev
worker: test: use constructor for tests
873be45
@turtleDev
app: worker: fix flush always timing out
e78a9d6
@turtleDev
app: move kafka specific log messages to kafka publisher package
3699a3a
@turtleDev
wip: worker metrics test
9f37c54
@turtleDev
worker: add tests against instrumented metrics
1612548
@turtleDev
worker: rename timeSource to Clock
cf724d7
@turtleDev
chore: remove client example
4e53510
@turtleDev
worker: use idiomatic null device for tests
5cc9f1a
@turtleDev
config: add tests for validation errors
1bbe539
@turtleDev
config: add test for valid configuration
03b8f6b
@turtleDev
config: add tests for internal prepare()
5f4c22b
@turtleDev
config: add tests for Walk()
8c4f37d
@turtleDev
config: kafka: add tests for ToConfigMap()
5d8a779
@turtleDev
config: kafka: add more tests for config map serializer
ad765fd
@turtleDev
config: add tests for internal helper functions
1ddad34
@turtleDev
refactor: extract Clock to a stand alone package
5132317
@turtleDev
collector: add tests for TimePushed mutation
e779dd0
@turtleDev
serialization: add tests for SerialiseProto
855968e
@turtleDev
deserialization: add tests for DeserializeFunc
fbc699c
@turtleDev
worker: add test for flush time out
977904e
@turtleDev
config: refactor ack value names
713f7c2
@turtleDev
tests: integration e2e tests results in coverage data
fa3d4c5
@turtleDev
ci: test: fix coverage merge failing
527b475
@turtleDev
ci: prospective fix for coverdata #1
84fabf3
@turtleDev
ci: prospective fix for coverdata #2
1d47e97
@turtleDev
ci: prospective fix for coverdata #3
af606d4
@turtleDev
ci: ignore proto package during tests
75d0807
@turtleDev
ci: remove docker image caching
aaa0571
@turtleDev
Revert "ci: ignore proto package during tests"
1038f46 
This reverts commit 75d0807.
@turtleDev
Revert "Revert "ci: ignore proto package during tests""
a39cad6 
This reverts commit 1038f46.
@turtleDev
Revert "Revert "Revert "ci: ignore proto package during tests"""
38b48f5 
This reverts commit a39cad6.
@turtleDev
publisher: log: move test event to root proto package
7f837bf
@turtleDev
ci: blacklist proto package during tests
0856073
@turtleDev
hack: publisher: log: use single spaced fields
5ffb846
@turtleDev
Revert "publisher: log: move test event to root proto package"
76e52ce 
This reverts commit 7f837bf.
@turtleDev
Revert "ci: blacklist proto package during tests"
98ac6dd 
This reverts commit 0856073.
@turtleDev
Revert "hack: publisher: log: use single spaced fields"
67797d9 
This reverts commit 5ffb846.
@turtleDev
Revert "Revert "hack: publisher: log: use single spaced fields""
986bf94 
This reverts commit 67797d9.
@turtleDev
ci: ignore proto folders when computing coverage
9672b13
@turtleDev
hack: publisher: log: use double spaced fields
a727bd7
@turtleDev
ci: fix coverage data merge
f01cc07
@turtleDev
ci: fix cover merge resulting in empty coverage file
9ec931c
@turtleDev
publish: kinesis: add tests for stream's existence check
d459c90
@turtleDev
misc: improve test case description
970bdcc
@turtleDev
publisher: kinesis: add tests for hitting rate limits and quotas
f78453a
@turtleDev
publisher: kinesis: refactor tests
e6fa9a1
@turtleDev
hygiene: reorganise package structure
c6a037a
@turtleDev
refactor: merge serialization and deserialization pkg into serde
d5309e5
@turtleDev
docs: fix broken dashboard link
bd14ba4
@turtleDev
serde: test: fix broken references
c32b74d
@turtleDev
refactor: integrate middleware package into server/rest
939727d
@turtleDev
Merge branch 'main' into reorg
ada541d
@coveralls
Copy link

Pull Request Test Coverage Report for Build 10834216238

Details

  • 36 of 41 (87.8%) changed or added relevant lines in 6 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.03%) to 81.269%
Changes Missing Coverage Covered Lines Changed/Added Lines %
server/rest/websocket/handler.go 7 8 87.5%
server/rest/service.go 11 15 73.33%
Totals Coverage Status
Change from base Build 10797042429: -0.03%
Covered Lines: 1896
Relevant Lines: 2333
💛 - Coveralls

@ravisuhag ravisuhag changed the title Reorganise Package Structure chore: reorganise Package Structure Sep 12, 2024
@ravisuhag ravisuhag merged commit a94d6f2 into main Sep 12, 2024
8 checks passed
@ravisuhag ravisuhag deleted the reorg branch September 12, 2024 20:35
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.
3 participants
@turtleDev @coveralls @ravisuhag