diff mbox series

[bug#62472,2/2] gnu: conan: Remove test_token_expired test.

Message ID 20230327081803.3832-2-rocktakey@gmail.com
State New
Headers
Series None |

Commit Message

ROCKTAKEY March 27, 2023, 8:18 a.m. UTC 
* gnu/packages/package-management.scm (conan): Remove test_token_expired test.
---
 gnu/packages/package-management.scm | 2 ++
 1 file changed, 2 insertions(+)

Comments

Maxim Cournoyer March 30, 2023, 8:13 p.m. UTC | #1 
Hello!

ROCKTAKEY <rocktakey@gmail.com> writes:

> * gnu/packages/package-management.scm (conan): Remove test_token_expired test.

Thanks for the submission.  I ended up attempting to update the package;
it was more work than I had hoped, but it works and reduced the number
of disabled tests.  See commit 7ceedc7df7d6f0d1209dee1eaa9c1dfe772ca0dd
("gnu: conan: Update to 2.0.2.").
ROCKTAKEY March 31, 2023, 10:25 a.m. UTC | #2 
Hi!

Thanks for your work. Updating to 2.0.2 is better.

But in my environment, some other tests are failed:

=========================== short test summary info 
============================
FAILED 
conans/test/functional/toolchains/cmake/test_cmake.py::LinuxTest::test_toolchain_linux_0_Debug
FAILED 
conans/test/integration/command_v2/list_test.py::TestListPrefs::test_list_pkg_ids[True]
FAILED 
conans/test/integration/command_v2/list_test.py::TestListPrefs::test_list_pkg_ids_all_rrevs[True]
FAILED 
conans/test/integration/command_v2/list_test.py::TestListPrefs::test_list_latest_prevs[True]
FAILED 
conans/test/integration/command_v2/list_test.py::TestListPrefs::test_list_all_prevs[True]
= 5 failed, 3016 passed, 321 skipped, 165 xfailed, 9 xpassed, 12 
warnings in 510.98s (0:08:30) =

--

ROCKTAKEY

On 2023/03/31 5:13, Maxim Cournoyer wrote:
> Hello!
>
> ROCKTAKEY <rocktakey@gmail.com> writes:
>
>> * gnu/packages/package-management.scm (conan): Remove test_token_expired test.
> Thanks for the submission.  I ended up attempting to update the package;
> it was more work than I had hoped, but it works and reduced the number
> of disabled tests.  See commit 7ceedc7df7d6f0d1209dee1eaa9c1dfe772ca0dd
> ("gnu: conan: Update to 2.0.2.").
>
Maxim Cournoyer March 31, 2023, 2:30 p.m. UTC | #3 
Hello,

ROCKTAKEY <rocktakey@gmail.com> writes:

> Hi!
>
> Thanks for your work. Updating to 2.0.2 is better.
>
> But in my environment, some other tests are failed:
>
> =========================== short test summary info
> ============================
> FAILED
> conans/test/functional/toolchains/cmake/test_cmake.py::LinuxTest::test_toolchain_linux_0_Debug
> FAILED
> conans/test/integration/command_v2/list_test.py::TestListPrefs::test_list_pkg_ids[True]
> FAILED
> conans/test/integration/command_v2/list_test.py::TestListPrefs::test_list_pkg_ids_all_rrevs[True]
> FAILED
> conans/test/integration/command_v2/list_test.py::TestListPrefs::test_list_latest_prevs[True]
> FAILED
> conans/test/integration/command_v2/list_test.py::TestListPrefs::test_list_all_prevs[True]
> = 5 failed, 3016 passed, 321 skipped, 165 xfailed, 9 xpassed, 12
> warnings in 510.98s (0:08:30) =

I was able to reproduce it for test_toolchain_linux_0_Debug, which I've
now skipped, but not for the other tests.  The build farm reproduced
them though [0]:

=========================== short test summary info ============================
FAILED conans/test/functional/toolchains/cmake/test_cmake.py::LinuxTest::test_toolchain_linux_0_Debug
FAILED conans/test/integration/command_v2/list_test.py::TestListPrefs::test_list_pkg_ids[True]
FAILED conans/test/integration/command_v2/list_test.py::TestListPrefs::test_list_pkg_ids_all_rrevs[True]
FAILED conans/test/integration/command_v2/list_test.py::TestListPrefs::test_list_latest_prevs[True]
FAILED conans/test/integration/command_v2/list_test.py::TestListPrefs::test_list_all_prevs[True]

[0]  https://ci.guix.gnu.org/build/666884/log/raw

I've reported the failure upstream and disabled these as well:

--8<---------------cut here---------------start------------->8---
modified   gnu/packages/package-management.scm
@@ -1209,7 +1209,7 @@ (define system #$(or (%current-target-system)
                 (setenv "CONFIG_SHELL" (which "sh"))
                 (setenv "PATH" (string-append (getenv "PATH") ":"
                                               #$output "/bin"))
-                (invoke "python" "-m" "pytest"
+                (invoke "python" "-m" "pytest" "-vv"
                         "-n" (number->string (parallel-job-count))
                         "-m" "not slow"
                         ;; Disable problematic tests.
@@ -1222,6 +1222,12 @@ (define system #$(or (%current-target-system)
                          ;; expected by the following test.
                          "and not pkg_config_path "
                          "and not compare " ;caused by newer node-semver?
+                         ;; This test hard-codes a compiler version.
+                         "and not test_toolchain "
+                         ;; The 'test_list' tests may fail
+                         ;; non-deterministically (see:
+                         ;; https://github.com/conan-io/conan/issues/13583).
+                         "and not test_list "
                          ;; These tests fail when Autoconf attempt to load a
                          ;; shared library in the same directory (see:
                          ;; https://github.com/conan-io/conan/issues/13577).
--8<---------------cut here---------------end--------------->8---

Hopefully it now builds on your side as well.  Thanks for the report!
ROCKTAKEY March 31, 2023, 4:57 p.m. UTC | #4 
It works well. Thanks!

On 2023/03/31 23:30, Maxim Cournoyer wrote:
> Hello,
>
> ROCKTAKEY <rocktakey@gmail.com> writes:
>
>> Hi!
>>
>> Thanks for your work. Updating to 2.0.2 is better.
>>
>> But in my environment, some other tests are failed:
>>
>> =========================== short test summary info
>> ============================
>> FAILED
>> conans/test/functional/toolchains/cmake/test_cmake.py::LinuxTest::test_toolchain_linux_0_Debug
>> FAILED
>> conans/test/integration/command_v2/list_test.py::TestListPrefs::test_list_pkg_ids[True]
>> FAILED
>> conans/test/integration/command_v2/list_test.py::TestListPrefs::test_list_pkg_ids_all_rrevs[True]
>> FAILED
>> conans/test/integration/command_v2/list_test.py::TestListPrefs::test_list_latest_prevs[True]
>> FAILED
>> conans/test/integration/command_v2/list_test.py::TestListPrefs::test_list_all_prevs[True]
>> = 5 failed, 3016 passed, 321 skipped, 165 xfailed, 9 xpassed, 12
>> warnings in 510.98s (0:08:30) =
> I was able to reproduce it for test_toolchain_linux_0_Debug, which I've
> now skipped, but not for the other tests.  The build farm reproduced
> them though [0]:
>
> =========================== short test summary info ============================
> FAILED conans/test/functional/toolchains/cmake/test_cmake.py::LinuxTest::test_toolchain_linux_0_Debug
> FAILED conans/test/integration/command_v2/list_test.py::TestListPrefs::test_list_pkg_ids[True]
> FAILED conans/test/integration/command_v2/list_test.py::TestListPrefs::test_list_pkg_ids_all_rrevs[True]
> FAILED conans/test/integration/command_v2/list_test.py::TestListPrefs::test_list_latest_prevs[True]
> FAILED conans/test/integration/command_v2/list_test.py::TestListPrefs::test_list_all_prevs[True]
>
> [0]  https://ci.guix.gnu.org/build/666884/log/raw
>
> I've reported the failure upstream and disabled these as well:
>
> --8<---------------cut here---------------start------------->8---
> modified   gnu/packages/package-management.scm
> @@ -1209,7 +1209,7 @@ (define system #$(or (%current-target-system)
>                   (setenv "CONFIG_SHELL" (which "sh"))
>                   (setenv "PATH" (string-append (getenv "PATH") ":"
>                                                 #$output "/bin"))
> -                (invoke "python" "-m" "pytest"
> +                (invoke "python" "-m" "pytest" "-vv"
>                           "-n" (number->string (parallel-job-count))
>                           "-m" "not slow"
>                           ;; Disable problematic tests.
> @@ -1222,6 +1222,12 @@ (define system #$(or (%current-target-system)
>                            ;; expected by the following test.
>                            "and not pkg_config_path "
>                            "and not compare " ;caused by newer node-semver?
> +                         ;; This test hard-codes a compiler version.
> +                         "and not test_toolchain "
> +                         ;; The 'test_list' tests may fail
> +                         ;; non-deterministically (see:
> +                         ;; https://github.com/conan-io/conan/issues/13583).
> +                         "and not test_list "
>                            ;; These tests fail when Autoconf attempt to load a
>                            ;; shared library in the same directory (see:
>                            ;; https://github.com/conan-io/conan/issues/13577).
> --8<---------------cut here---------------end--------------->8---
>
> Hopefully it now builds on your side as well.  Thanks for the report!
>
diff mbox series

Patch

diff --git a/gnu/packages/package-management.scm b/gnu/packages/package-management.scm
index e5048c95d6..bcd072a442 100644
--- a/gnu/packages/package-management.scm
+++ b/gnu/packages/package-management.scm
@@ -1219,6 +1219,8 @@  (define system ,(or (%current-target-system)
                         ;; https://lore.kernel.org/git/xmqq4jw1uku5.fsf@gitster.g/T/#u
                         "and not test_submodule "
                         "and not test_clone_submodule_git "
+                        ;; FIXME: Sometimes failed due to logging in problem on "conan upload"
+                        "and not test_token_expired "
                         (if (not (string-prefix? "x86_64" system))
                             ;; These tests either assume the machine is
                             ;; x86_64, or require a cross-compiler to target