#26 Output CI log to stdout
Opened 3 years ago by fujiwara. Modified 3 years ago
rpms/ fujiwara/ibus test3  into  rawhide

file modified
+26 -5
@@ -1,6 +1,6 @@ 

- From ec6b251149bb5ca79c73f7bdbded55d72a44b3a1 Mon Sep 17 00:00:00 2001

+ From b3b90f0e467ade81165d67638ca7c2541963a879 Mon Sep 17 00:00:00 2001

  From: fujiwarat <takao.fujiwara1@gmail.com>

- Date: Sat, 20 Mar 2021 11:40:50 +0900

+ Date: Sat, 20 Mar 2021 20:28:42 +0900

  Subject: [PATCH] src/tests: Change default session gnome-session to mutter

  

  gnome-shell 40 now shows the overview mode by login and
@@ -9,13 +9,18 @@ 

  The default may be changed to twm if mutter also will be changed

  not to accept the application focus in the future.

  

+ Disable Tour dialog which prevent test application from getting the

+ input focus.

+ 

+ Don't output FAIL if the actual failure is 0 for Fedora CI.

+ 

  BUG=https://discourse.gnome.org/t/focus-on-autostart-application-by-login/5863

  ---

-  src/tests/ibus-desktop-testing-runner.in | 22 ++++++++++++++++++----

-  1 file changed, 18 insertions(+), 4 deletions(-)

+  src/tests/ibus-desktop-testing-runner.in | 31 +++++++++++++++++++-----

+  1 file changed, 25 insertions(+), 6 deletions(-)

  

  diff --git a/src/tests/ibus-desktop-testing-runner.in b/src/tests/ibus-desktop-testing-runner.in

- index 4232c549..92ae14ba 100755

+ index 4232c549..23e3411e 100755

  --- a/src/tests/ibus-desktop-testing-runner.in

  +++ b/src/tests/ibus-desktop-testing-runner.in

  @@ -4,7 +4,7 @@
@@ -82,6 +87,22 @@ 

       fi

   }

   

+ @@ -360,8 +374,13 @@ EOF_RUNNER

+          ;;

+      esac

+      echo ""

+ -    print_log -e "${GREEN}PASS${NC}: $pass"

+ -    print_log -e "${RED}FAIL${NC}: $fail"

+ +    # Fedora CI assumes the test is failed even if $fail is 0.

+ +    if [ $pass -ne 0 ] ; then

+ +        print_log -e "${GREEN}PASS${NC}: $pass"

+ +    fi

+ +    if [ $fail -ne 0 ] ; then

+ +        print_log -e "${RED}FAIL${NC}: $fail"

+ +    fi

+      echo ""

+      if [ $TEST_LOG_STDOUT -eq 1 ] ; then

+          cat $TEST_LOG

  -- 

  2.28.0

  

file modified
+7 -1
@@ -38,7 +38,7 @@ 

  

  Name:           ibus

  Version:        1.5.24

- Release:        2%{?dist}

+ Release:        4%{?dist}

  Summary:        Intelligent Input Bus for Linux OS

  License:        LGPLv2+

  URL:            https://github.com/ibus/%name/wiki
@@ -488,6 +488,12 @@ 

  %{_datadir}/installed-tests/ibus

  

  %changelog

+ * Wed Apr 21 2021 Takao Fujiwara <tfujiwar@redhat.com> - 1.5.24-4

+ - Merge the latest rawhide

+ 

+ * Sat Mar 20 2021 Takao Fujiwara <tfujiwar@redhat.com> - 1.5.24-3

+ - Don't output FAIL if the actual failure is 0 for Fedora CI

+ 

  * Sat Mar 20 2021 Takao Fujiwara <tfujiwar@redhat.com> - 1.5.24-2

  - Change default session to mutter from gnome in desktop-testing

  

@@ -76,30 +76,35 @@ 

        if [ $? -eq 0 ]; then

            status="PASS: frame"

        fi

-       echo "${status} $TEST" >> {{ remote_artifacts }}/test.log

+       echo "${status}" >> {{ remote_artifacts }}/test.log

+       echo "#### {{ remote_artifacts }}/{{ installed_test_name }}.log"

+       if [ -f {{ remote_artifacts }}/{{ installed_test_name }}.log ] ; then

+           cat {{ remote_artifacts }}/{{ installed_test_name }}.log

+       fi

+       echo "#"

+       echo "#### {{ remote_artifacts }}/test.log"

+       if [ -f {{ remote_artifacts }}/test.log ] ; then

+           cat {{ remote_artifacts }}/test.log

+       fi

+       echo "#"

  

    - name: Check the results

      shell: |

          IS_RAWHIDE=`grep -i rawhide /etc/fedora-release`

          if [ x"$IS_RAWHIDE" != x ] ; then

-             echo PASS

              exit 0

          fi

          log="{{ remote_artifacts }}/test.log"

          if [ ! -f $log ] ; then

              echo ERROR

-             exit 1

          else

              FAIL=`grep "^FAIL: " $log | grep -v 'FAIL: 0$'`

              if [ x"$FAIL" != x ] ; then

                  echo ERROR

-                 exit 1

-             else

-                 echo PASS

              fi

          fi

      register: test_fails

-     #failed_when: False

+     failed_when: False

  

    - name: Set role result

      set_fact:

no initial comment

1 new commit added

  • Test2
3 years ago

1 new commit added

  • Test3
3 years ago

1 new commit added

  • Test4
3 years ago

1 new commit added

  • Test5
3 years ago

1 new commit added

  • Test6
3 years ago

it is not working yet, will need more fixes ...

@mvadkert I can see the result is succeeded with the small test:
https://src.fedoraproject.org/rpms/input-pad/pull-request/4

I guess this ibus issue can be fixed after I delete the "FAIL: 0" test.

1 new commit added

  • Merge the latest rawhide
3 years ago