Skip to content

fix(cts): rename test #5030

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
Jun 27, 2025
Merged

fix(cts): rename test #5030

merged 1 commit into from
Jun 27, 2025

Conversation

kai687
Copy link
Contributor

@kai687 kai687 commented Jun 25, 2025

🧭 What and Why

The testName "set_searchable_attributes" occurred twice.
One of them is actually for attributesForFaceting.

🎟 JIRA Ticket:

Changes included:

  • Rename test to set_attributes_for_faceting

🧪 Test

@algolia-bot
Copy link
Collaborator

algolia-bot commented Jun 25, 2025

✔️ Code generated!

Name Link
🪓 Triggered by 1fe654dbd1d5932f12db29eb2ac972796d7582bf
🍃 Generated commit a6eabb851112bb0eb7af9ed35afb5ba0a971b928
🌲 Generated branch generated/fix/snippets/searchable-attributes
📊 Benchmark results

Benchmarks performed on the method using a mock server, the results might not reflect the real-world performance.

Language Req/s
go 1771
javascript 1684
csharp 1451
php 1431
java 1076
python 1071
ruby 830
swift 777
scala 23

@kai687 kai687 changed the title fix: rename test fix(cts): rename test Jun 25, 2025
@kai687 kai687 marked this pull request as ready for review June 25, 2025 16:57
@kai687 kai687 requested a review from a team as a code owner June 25, 2025 16:57
@kai687 kai687 requested review from millotp and shortcuts June 25, 2025 16:57
@millotp millotp merged commit 1a9b067 into main Jun 27, 2025
29 of 30 checks passed
@millotp millotp deleted the fix/snippets/searchable-attributes branch June 27, 2025 10:47
algolia-bot added a commit that referenced this pull request Jun 27, 2025
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.

3 participants