Skip to content

Commit c55bd54

Browse files
committed
add failing tests to show that we cannot filter by e.g. notes_count_eq
1 parent a28f6d4 commit c55bd54

2 files changed

Lines changed: 20 additions & 0 deletions

File tree

spec/dummy.rb

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@
1818
create_table :users, force: true do |t|
1919
t.string :first_name
2020
t.string :last_name
21+
t.integer :notes_count, default: 0
2122
t.timestamps
2223
end
2324

spec/filtering_spec.rb

Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -85,6 +85,25 @@
8585
end
8686
end
8787

88+
context 'returns users by counter_cache' do
89+
let(:params) do
90+
second_user.update(notes_count: 1)
91+
{
92+
filter: { notes_count_eq: 1 }
93+
}
94+
end
95+
96+
fit do
97+
expect(first_user.notes_count).to eq(0)
98+
expect(second_user.notes_count).to eq(1)
99+
expect(third_user.notes_count).to eq(0)
100+
101+
expect(response).to have_http_status(:ok)
102+
expect(response_json['data'].size).to eq(1)
103+
expect(response_json['data'][0]).to have_id(second_user.id.to_s)
104+
end
105+
end
106+
88107
context 'returns sorted users by notes' do
89108
let(:params) do
90109
{ sort: '-notes_created_at' }

0 commit comments

Comments
 (0)