Skip to content

Conversation

@bruce-hong-glean
Copy link
Contributor

@bruce-hong-glean bruce-hong-glean commented Aug 13, 2025

Description

The PR introducing false_allow_templates added tests that try to test for the float and integer types which aren't supported. This PR removes the invalid tests.

Check List

  • Functionality includes testing.
  • API changes companion pull request created, if applicable.
  • Public documentation issue/PR created, if applicable.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

bruce-hong-glean and others added 21 commits July 24, 2025 10:01
Update tests

Tests

Tests

Cleanup

Tested, cleanup

Cleanup

Signed-off-by: Bruce Hong <bruce.hong@glean.com>
Signed-off-by: Bruce Hong <bruce.hong@glean.com>
Signed-off-by: Bruce Hong <bruce.hong@glean.com>
Signed-off-by: Bruce Hong <bruce.hong@glean.com>
Signed-off-by: Bruce Hong <bruce.hong@glean.com>
Signed-off-by: Bruce Hong <bruce.hong@glean.com>
Signed-off-by: Bruce Hong <bruce.hong@glean.com>
Signed-off-by: Bruce Hong <bruce.hong@glean.com>
Signed-off-by: Bruce Hong <bruce.hong@glean.com>
Signed-off-by: Bruce Hong <bruce.hong@glean.com>
Signed-off-by: gaobinlong <gbinlong@amazon.com>
Signed-off-by: gaobinlong <gbinlong@amazon.com>
Signed-off-by: Bruce Hong <bruce.hong@glean.com>
Signed-off-by: Bruce Hong <bruce.hong@glean.com>
Signed-off-by: Bruce Hong <bruce.hong@glean.com>
Signed-off-by: Bruce Hong <bruce.hong@glean.com>
Signed-off-by: Bruce Hong <bruce.hong@glean.com>
Signed-off-by: Bruce Hong <bruce.hong@glean.com>
@bruce-hong-glean bruce-hong-glean changed the title Remove invalid type testing for false_allow_templates Remove invalid type tests for false_allow_templates Aug 13, 2025
@bruce-hong-glean bruce-hong-glean marked this pull request as ready for review August 13, 2025 19:34
@bruce-hong-glean bruce-hong-glean requested a review from a team as a code owner August 13, 2025 19:34
@bruce-hong-glean bruce-hong-glean marked this pull request as draft August 13, 2025 19:35
Signed-off-by: Bruce Hong <bruce.hong@glean.com>
@github-actions
Copy link
Contributor

❌ Gradle check result for 1dbfc1e: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

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.

2 participants