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

TestJitBuilderAPIGenerator failure: no attribute assertRegexpMatches #7181

Open
babsingh opened this issue Nov 13, 2023 · 3 comments · May be fixed by #7279
Open

TestJitBuilderAPIGenerator failure: no attribute assertRegexpMatches #7181

babsingh opened this issue Nov 13, 2023 · 3 comments · May be fixed by #7279
Labels
comp:jitbuilder

Comments

@babsingh
Copy link
Contributor

https://dev.azure.com/eclipse-omr/ea4519db-b27e-4d19-a971-f01491f803e3/_apis/build/builds/8241/logs/67

2023-11-10T22:20:05.5840210Z 2: ..................................................................................................E...EEEEEEEEEEEEEEEE.............EE.EEsEEsEEs...
2023-11-10T22:20:05.5841430Z 2: ======================================================================
2023-11-10T22:20:05.5842770Z 2: ERROR: test_generate_allocator_decl_1 (test.cppgentests.CppGeneratorTest.test_generate_allocator_decl_1)
2023-11-10T22:20:05.5844840Z 2: ----------------------------------------------------------------------
2023-11-10T22:20:05.5845760Z 2: Traceback (most recent call last):
2023-11-10T22:20:05.5847120Z 2:   File "/Users/runner/work/1/s/jitbuilder/apigen/test/cppgentests.py", line 245, in test_generate_allocator_decl_1
2023-11-10T22:20:05.5848850Z 2:     self.assertRegexpMatches(self.generator.generate_allocator_decl(class_desc),
2023-11-10T22:20:05.5850090Z 2:     ^^^^^^^^^^^^^^^^^^^^^^^^
2023-11-10T22:20:05.5851190Z 2: AttributeError: 'CppGeneratorTest' object has no attribute 'assertRegexpMatches'
2023-11-10T22:20:05.5852180Z 2: 

...

2023-11-10T22:20:05.6070130Z 2: FAILED (errors=25, skipped=3)
2023-11-10T22:20:05.6070920Z 2: warning: The package 'jsonschema' is not installed so certain tests will be skipped
2023-11-10T22:20:05.6072030Z 32/49 Test  #2: TestJitBuilderAPIGenerator ..........***Failed    0.94 sec
@babsingh
Copy link
Contributor Author

In my quick scan of the previous PR builds (https://dev.azure.com/eclipse-omr/omr/_build?definitionId=1&_a=summary), I found that the failure is also seen in #7046. fyi @hzongaro @jdmpapin

@babsingh babsingh mentioned this issue Nov 13, 2023
Merged
@hzongaro
Copy link
Member

In a build I found where that test passed, I see this:

7: Working Directory: /Users/runner/work/1/s/build/jitbuilder/release
27: Test timeout computed to be: 10000000
2: ................................................................................................../Users/runner/work/1/s/jitbuilder/apigen/test/cppgentests.py:245: DeprecationWarning: Please use assertRegex instead.
2:   self.assertRegexpMatches(self.generator.generate_allocator_decl(class_desc),

@babsingh
Copy link
Contributor Author

babsingh commented Nov 14, 2023 •
edited

In the Azure pipeline, macOS-latest is used for the VM image, which probably has the latest Python version where assertRegexpMatches no longer exists. If JitBuilder has a specific Python version requirement, then it can be included in azure-pipelines.yml to resolve this issue. fyi @AdamBrousseau

@mingweiarthurli mingweiarthurli mentioned this issue Nov 16, 2023
Merged
@a7ehuo a7ehuo mentioned this issue Nov 21, 2023
Merged
1 task
@BradleyWood BradleyWood mentioned this issue Nov 21, 2023
Merged
@babsingh babsingh mentioned this issue Nov 30, 2023
Merged
@hzongaro hzongaro mentioned this issue Dec 11, 2023
Merged
@babsingh babsingh mentioned this issue Dec 12, 2023
Merged
@knn-k knn-k mentioned this issue Dec 13, 2023
Merged
This was referenced Dec 15, 2023
Merged
Merged
@dsouzai dsouzai mentioned this issue Dec 18, 2023
Merged
This was referenced Dec 19, 2023
Merged
Merged
@singh264 singh264 mentioned this issue Jan 2, 2024
Merged
@dsouzai dsouzai mentioned this issue Jan 3, 2024
Merged
@hzongaro hzongaro mentioned this issue Jan 7, 2024
Merged
@babsingh babsingh mentioned this issue Jan 8, 2024
Merged
@singh264 singh264 mentioned this issue Jan 11, 2024
Merged
@hzongaro hzongaro mentioned this issue Jan 16, 2024
Merged
@knn-k knn-k mentioned this issue Jan 18, 2024
Merged
@a7ehuo a7ehuo mentioned this issue Jan 19, 2024
Merged
@singh264 singh264 mentioned this issue Jan 19, 2024
Merged
@jdmpapin jdmpapin mentioned this issue Jan 19, 2024
Merged
@BradleyWood BradleyWood mentioned this issue Jan 19, 2024
Draft
@a7ehuo a7ehuo mentioned this issue Jan 19, 2024
Merged
1 task
@VermaSh VermaSh mentioned this issue Jan 20, 2024
Merged
@0xdaryl 0xdaryl linked a pull request Mar 1, 2024 that will close this issue
Draft
@babsingh babsingh mentioned this issue Mar 2, 2024
Merged
@jmesyou jmesyou mentioned this issue Mar 5, 2024
Merged
@Spencer-Comin Spencer-Comin mentioned this issue Mar 18, 2024
Open
This was referenced Mar 19, 2024
Merged
Merged
Merged
Merged
@dsouzai dsouzai mentioned this issue Mar 21, 2024
Merged
@jmesyou jmesyou mentioned this issue Mar 22, 2024
Merged
This was referenced Mar 22, 2024
Merged
Merged
Merged
@BradleyWood BradleyWood mentioned this issue Apr 2, 2024
Merged
@jdmpapin jdmpapin mentioned this issue Apr 3, 2024
Merged
@Spencer-Comin Spencer-Comin mentioned this issue Apr 4, 2024
Open
@0xdaryl 0xdaryl mentioned this issue Apr 5, 2024
Merged
@knn-k knn-k mentioned this issue Apr 11, 2024
Merged
@a7ehuo a7ehuo mentioned this issue Apr 16, 2024
Merged
This was referenced Apr 18, 2024
Merged
Merged
@babsingh babsingh mentioned this issue Apr 22, 2024
Merged
@hzongaro hzongaro mentioned this issue Apr 23, 2024
Merged
@Spencer-Comin Spencer-Comin mentioned this issue Apr 25, 2024
Open
@knn-k knn-k mentioned this issue Apr 25, 2024
Merged
@babsingh babsingh mentioned this issue May 1, 2024
Merged
@a7ehuo a7ehuo mentioned this issue May 10, 2024
Open
@R2steven R2steven mentioned this issue May 10, 2024
Merged
This was referenced May 14, 2024
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
comp:jitbuilder
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Rename deprecated Python unittest function assertRegexpMatches 0xdaryl/omr
2 participants
@babsingh @hzongaro