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.

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

Flagsets merge to dev #501

Merged
merged 39 commits into from
Dec 7, 2023
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
39 commits
Select commit Hold shift + click to select a range
2c32af6
Merge pull request #480 from splitio/development
sanzmauro Jul 18, 2023
e9129d2
updated engine.api.splits class
chillaq Oct 20, 2023
fdc88cc
added exception for 409 status code
chillaq Oct 21, 2023
b4b43dc
added flag_set_filter config param and validation
chillaq Oct 23, 2023
70b3436
updated error code
chillaq Oct 24, 2023
3908a97
Merge pull request #483 from splitio/flagsets-engine-api
chillaq Oct 24, 2023
bd9e669
fixed regex
chillaq Oct 24, 2023
b838d78
updated telemetry classes
chillaq Oct 24, 2023
f1b88f0
Merge pull request #484 from splitio/flagset-client-config
chillaq Oct 27, 2023
0ce3a2a
Merge pull request #485 from splitio/flagset-telemetry
chillaq Oct 27, 2023
06db54c
update client class and fixed tests
chillaq Nov 3, 2023
94b350c
added updates from other prs
chillaq Nov 3, 2023
b6f302f
polishing
chillaq Nov 6, 2023
27af19d
Merge pull request #488 from splitio/flagset-client-splitclient
chillaq Nov 10, 2023
d0a03e2
refactor evaluator with splitclient
chillaq Nov 10, 2023
554508e
fixed typo
chillaq Nov 10, 2023
1bc870f
polishing with test
chillaq Nov 13, 2023
eb72e1c
Fixed rubocop offenses
chillaq Nov 14, 2023
5be6722
Merge pull request #493 from splitio/flagset-evaluator-refactor
chillaq Nov 16, 2023
f87ab3d
Updated all specs plus minor fixes
chillaq Nov 17, 2023
1d8864c
Merge pull request #495 from splitio/flagset-specs
chillaq Nov 27, 2023
da38227
Added redis flag set repo and validation cleanup
chillaq Nov 27, 2023
7ac3a85
used redis piplined and updated flag_set repositories to fetch array
chillaq Nov 28, 2023
31b59f6
Merge pull request #497 from splitio/flagset-redis-cache
chillaq Nov 29, 2023
6a228af
Added support for SSE and minor fix in updating split repository
chillaq Nov 29, 2023
5c5be71
Merge pull request #498 from splitio/flagset-sse-splitworker
chillaq Nov 30, 2023
3f753bb
removed SortedSet and inlcude 'set'
chillaq Dec 1, 2023
c04381b
Merge pull request #499 from splitio/flagset-remove-sortedset
chillaq Dec 4, 2023
dd99e35
Added sets to manager split view dict
chillaq Dec 4, 2023
642ae8e
return empty list when nil sets
chillaq Dec 4, 2023
e7c82f5
Merge pull request #500 from splitio/flagsets-manager-splitview
sanzmauro Dec 4, 2023
dc9f3c0
merge flagset build
chillaq Dec 7, 2023
9d9d4fb
Merge branch 'development' into flagsets-merge-to-dev
chillaq Dec 7, 2023
818d219
Update CHANGES.txt
chillaq Dec 7, 2023
b5b549d
fix rubocop offense
chillaq Dec 7, 2023
3c0e2e3
cleanup
chillaq Dec 7, 2023
c468069
fixed engine_spec for ruby 3.x
chillaq Dec 7, 2023
bec1e81
fixed typo in telemetry record latency
chillaq Dec 7, 2023
c8e27a9
fixed typo and cleanup
chillaq Dec 7, 2023
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
cleanup
  • Loading branch information
chillaq committed Dec 7, 2023
commit 3c0e2e3e1abba677ad48c06a0f2a7c12cd13e3d9
2 changes: 1 addition & 1 deletion lib/splitclient-rb/managers/split_manager.rb
Original file line number Diff line number Diff line change
Expand Up @@ -106,7 +106,7 @@ def build_split_view(name, split)
treatments: treatments,
change_number: split[:changeNumber],
configs: split[:configurations] || {},
sets: split[:sets] || []
sets: split[:sets] || [],
default_treatment: split[:defaultTreatment]
}
end
Expand Down
18 changes: 12 additions & 6 deletions spec/splitclient/manager_localhost_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,8 @@
killed: false,
name: 'local_feature',
traffic_type_name: nil,
treatments: ['local_treatment'] }
treatments: ['local_treatment'],
sets: [] }
end

let(:split_views) do
Expand All @@ -30,14 +31,16 @@
killed: false,
name: 'local_feature',
traffic_type_name: nil,
treatments: ['local_treatment'] },
treatments: ['local_treatment'],
sets: [] },
{ change_number: nil,
configs: { local_treatment2: nil },
default_treatment: "control_treatment",
killed: false,
name: 'local_feature2',
traffic_type_name: nil,
treatments: ['local_treatment2'] }]
treatments: ['local_treatment2'],
sets: [] }]
end

it 'validates the calling manager.splits returns the offline data' do
Expand Down Expand Up @@ -66,7 +69,8 @@
killed: false,
name: 'multiple_keys_feature',
traffic_type_name: nil,
treatments: %w[off on] },
treatments: %w[off on],
sets: [] },
{ change_number: nil,
configs: {
on: '{"desc":"this applies only to ON and only for john_doe. The rest will receive OFF"}',
Expand All @@ -76,7 +80,8 @@
killed: false,
name: 'single_key_feature',
traffic_type_name: nil,
treatments: %w[on off] },
treatments: %w[on off],
sets: [] },
{ change_number: nil,
configs: {
off: '{"desc":"this applies only to OFF treatment"}'
Expand All @@ -85,7 +90,8 @@
killed: false,
name: 'no_keys_feature',
traffic_type_name: nil,
treatments: %w[off] }]
treatments: %w[off],
sets: [] }]
end

it 'returns split_names' do
Expand Down
Loading