[Patchew-devel] [PATCH 0/4] tests: Fix repeated error reports

Fam Zheng posted 4 patches 5 years, 8 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/patchew-ci tags/patchew/20180802081422.25982-1-famz@redhat.com
manage.py                                     |   5 +-
mods/tags.py                                  |  15 +++--
mods/testing.py                               |  18 ++++--
.../0025-foo-patch-add-reviewed-by.mbox.gz    | Bin 0 -> 1104 bytes
tests/data/0026-new-file.mbox.gz              | Bin 0 -> 949 bytes
.../data/0027-new-file-based-on-reply.mbox.gz | Bin 0 -> 863 bytes
tests/test_testing.py                         |  61 ++++++++++++++++++
7 files changed, 88 insertions(+), 11 deletions(-)
create mode 100644 tests/data/0025-foo-patch-add-reviewed-by.mbox.gz
create mode 100644 tests/data/0026-new-file.mbox.gz
create mode 100644 tests/data/0027-new-file-based-on-reply.mbox.gz
[Patchew-devel] [PATCH 0/4] tests: Fix repeated error reports
Posted by Fam Zheng 5 years, 8 months ago
This series got three duplicated checkpatch error reports:

https://patchew.org/QEMU/20180801102944.23457-1-laurent@vivier.eu/

because the git result status progressively goes along:

           import              apply               rev-by[1]
    None ----------> PENDING ----------> SUCCESS ------------> PENDING
           apply               rev-by[2]             apply
         ----------> SUCCESS ----------> PENDING ------------> SUCCESS

and actually every "-> SUCCESS" transition triggers testing results reset (and
rerun).

rev-by tags needn't trigger rerun of tests. This series fixes it. Meanwhile,
handle based-on updates differently (no longer indirectly depending on git).

Fam Zheng (4):
  manage.py: Run tests/ if no test specified
  tags: Improve TagsUpdate event
  testing: Don't reset all tests upon git result update
  tests: Add testing cases for tags update

 manage.py                                     |   5 +-
 mods/tags.py                                  |  15 +++--
 mods/testing.py                               |  18 ++++--
 .../0025-foo-patch-add-reviewed-by.mbox.gz    | Bin 0 -> 1104 bytes
 tests/data/0026-new-file.mbox.gz              | Bin 0 -> 949 bytes
 .../data/0027-new-file-based-on-reply.mbox.gz | Bin 0 -> 863 bytes
 tests/test_testing.py                         |  61 ++++++++++++++++++
 7 files changed, 88 insertions(+), 11 deletions(-)
 create mode 100644 tests/data/0025-foo-patch-add-reviewed-by.mbox.gz
 create mode 100644 tests/data/0026-new-file.mbox.gz
 create mode 100644 tests/data/0027-new-file-based-on-reply.mbox.gz

-- 
2.17.1

_______________________________________________
Patchew-devel mailing list
Patchew-devel@redhat.com
https://www.redhat.com/mailman/listinfo/patchew-devel