Skip to content
Snippets Groups Projects

package/tests/test_utils: Remove code duplication

Closed package/tests/test_utils: Remove code duplication
generated-differential-D6254-source into generated-differential-D6254-target
Closed Antoine Lambert requested to merge generated-differential-D6254-source into generated-differential-D6254-target

Those tests implementation are quite similar so let's put generic test code in a function and use some global variables.

Depends on !223 (closed)

Migrated from D6254 (view on Phabricator)

Merge request reports

Closed by Phabricator Migration userPhabricator Migration user 3 years ago (Sep 16, 2021 8:48am UTC)

Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Software Heritage Continuous Integration for public repositories
    Software Heritage Continuous Integration for public repositories @swh-public-ci ·

    Build is green

    Patch application report for D6254 (id=22643)

    Could not rebase; Attempt merge onto d5e54a5e...

    Updating d5e54a5..6195214
Fast-forward
 swh/loader/package/tests/test_utils.py | 116 ++++++++++++++++-----------------
 swh/loader/package/utils.py            |   8 +++
 2 files changed, 66 insertions(+), 58 deletions(-)
    Changes applied before test
    commit 619521480d514f73c5827d59927de560de81a61d
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Tue Sep 14 13:38:34 2021 +0200

    package/tests/test_utils: Remove code duplication

commit a60ba533470fd4d46a753f8bc765ad706b071384
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Tue Sep 14 13:22:37 2021 +0200

    package/utils: Try to extract download filename from response headers
    
    Some URLs for downloading a file do not contain any filename but
    rather provide it in the "content-disposition" response header.
    
    So ensure to extract the filename from that response header when
    available to avoid possible file processing issues afterwards.

commit 0e9cd127168e3fdd7b3ee78ccb6ea55cceb3f94f
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Tue Sep 14 12:08:53 2021 +0200

    package/utils: Use download response URL to extract filename
    
    requests follows URL redirection by default for GET requests so
    update input URL to response one to ensure correct filename will
    be extracted from it.

    See https://jenkins.softwareheritage.org/job/DLDBASE/job/tests-on-diff/547/ for more details.

  • Antoine Lambert
    Antoine Lambert @anlambert ·
    Author Maintainer

    Rebase

  • Software Heritage Continuous Integration for public repositories
    Software Heritage Continuous Integration for public repositories @swh-public-ci ·

    Build is green

    Patch application report for D6254 (id=22647)

    Could not rebase; Attempt merge onto d5e54a5e...

    Updating d5e54a5..88fc559
Fast-forward
 swh/loader/package/tests/test_utils.py | 117 +++++++++++++++++----------------
 swh/loader/package/utils.py            |  14 ++++
 2 files changed, 73 insertions(+), 58 deletions(-)
    Changes applied before test
    commit 88fc559c3ce8280282f3ccab46f39ea9e8405c8f
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Tue Sep 14 13:38:34 2021 +0200

    package/tests/test_utils: Remove code duplication
    
    Those tests implementation are quite similar so let's put generic
    test code in a function and use some global variables.

commit a11695f9cebc49cdbe1972de8ed89b98b5fd1311
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Tue Sep 14 13:22:37 2021 +0200

    package/utils: Try to extract download filename from response headers
    
    Some URLs for downloading a file do not contain any filename but
    rather provide it in the "content-disposition" response header.
    
    So ensure to extract the filename from that response header when
    available to avoid possible file processing issues afterwards.

commit 0e9cd127168e3fdd7b3ee78ccb6ea55cceb3f94f
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Tue Sep 14 12:08:53 2021 +0200

    package/utils: Use download response URL to extract filename
    
    requests follows URL redirection by default for GET requests so
    update input URL to response one to ensure correct filename will
    be extracted from it.

    See https://jenkins.softwareheritage.org/job/DLDBASE/job/tests-on-diff/549/ for more details.

  • Antoine R. Dumont mentioned in merge request !223 (closed)

    mentioned in merge request !223 (closed)

  • Antoine R. Dumont
    Antoine R. Dumont @ardumont ·
    Maintainer

    Merge request was accepted

  • Antoine R. Dumont approved this merge request

    approved this merge request

  • Antoine Lambert
    Antoine Lambert @anlambert ·
    Author Maintainer

    Rebase

  • Software Heritage Continuous Integration for public repositories
    Software Heritage Continuous Integration for public repositories @swh-public-ci ·

    Build is green

    Patch application report for D6254 (id=22673)

    Could not rebase; Attempt merge onto d5e54a5e...

    Updating d5e54a5..fa9ee78
Fast-forward
 swh/loader/package/tests/test_utils.py | 140 +++++++++++++++++++--------------
 swh/loader/package/utils.py            |  24 ++++++
 2 files changed, 107 insertions(+), 57 deletions(-)
    Changes applied before test
    commit fa9ee788d28884f2d52e63ca003c2f286498c0c9
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Tue Sep 14 13:38:34 2021 +0200

    package/tests/test_utils: Remove code duplication
    
    Those tests implementation are quite similar so let's put generic
    test code in a function and use some global variables.

commit 716951dc7284968a49ed028bd33697da6b818675
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Tue Sep 14 13:22:37 2021 +0200

    package/utils: Try to extract download filename from response headers
    
    Some URLs for downloading a file do not contain any filename but
    rather provide it in the "content-disposition" response header.
    
    So ensure to extract the filename from that response header when
    available to avoid possible file processing issues afterwards.

commit 0e9cd127168e3fdd7b3ee78ccb6ea55cceb3f94f
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Tue Sep 14 12:08:53 2021 +0200

    package/utils: Use download response URL to extract filename
    
    requests follows URL redirection by default for GET requests so
    update input URL to response one to ensure correct filename will
    be extracted from it.

    See https://jenkins.softwareheritage.org/job/DLDBASE/job/tests-on-diff/551/ for more details.

  • Antoine Lambert
    Antoine Lambert @anlambert ·
    Author Maintainer

    Rebase

  • Software Heritage Continuous Integration for public repositories
    Software Heritage Continuous Integration for public repositories @swh-public-ci ·

    Build is green

    Patch application report for D6254 (id=22686)

    Could not rebase; Attempt merge onto d5e54a5e...

    Updating d5e54a5..78bdbaa
Fast-forward
 swh/loader/package/tests/test_utils.py | 140 +++++++++++++++++++--------------
 swh/loader/package/utils.py            |  24 ++++++
 2 files changed, 107 insertions(+), 57 deletions(-)
    Changes applied before test
    commit 78bdbaa471e21943310d57a157fd78562dca4fe6
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Tue Sep 14 13:38:34 2021 +0200

    package/tests/test_utils: Remove code duplication
    
    Those tests implementation are quite similar so let's put generic
    test code in a function and use some global variables.

commit cc73c630bd9be291705e34938cd146696106af29
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Tue Sep 14 13:22:37 2021 +0200

    package/utils: Try to extract download filename from response headers
    
    Some URLs for downloading a file do not contain any filename but
    rather provide it in the "content-disposition" response header.
    
    So ensure to extract the filename from that response header when
    available to avoid possible file processing issues afterwards.

commit 0e9cd127168e3fdd7b3ee78ccb6ea55cceb3f94f
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Tue Sep 14 12:08:53 2021 +0200

    package/utils: Use download response URL to extract filename
    
    requests follows URL redirection by default for GET requests so
    update input URL to response one to ensure correct filename will
    be extracted from it.

    See https://jenkins.softwareheritage.org/job/DLDBASE/job/tests-on-diff/553/ for more details.

  • Antoine Lambert
    Antoine Lambert @anlambert ·
    Author Maintainer

    Rebase

  • Software Heritage Continuous Integration for public repositories
    Software Heritage Continuous Integration for public repositories @swh-public-ci ·

    Build is green

    Patch application report for D6254 (id=22731)

    Could not rebase; Attempt merge onto 73299984...

    Updating 7329998..f4fe1f3
Fast-forward
 swh/loader/package/tests/test_utils.py | 140 +++++++++++++++++++--------------
 swh/loader/package/utils.py            |  24 ++++++
 2 files changed, 107 insertions(+), 57 deletions(-)
    Changes applied before test
    commit f4fe1f30bbe7e3f005cc4e31db91e2ab96b76025
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Tue Sep 14 13:38:34 2021 +0200

    package/tests/test_utils: Remove code duplication
    
    Those tests implementation are quite similar so let's put generic
    test code in a function and use some global variables.

commit 1bd1827fa27466365bee22d9e10d759667242990
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Tue Sep 14 13:22:37 2021 +0200

    package/utils: Try to extract download filename from response headers
    
    Some URLs for downloading a file do not contain any filename but
    rather provide it in the "content-disposition" response header.
    
    So ensure to extract the filename from that response header when
    available to avoid possible file processing issues afterwards.

commit 049c41f6144d9cf80462c82e4f03f2a25fc45ebb
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Tue Sep 14 12:08:53 2021 +0200

    package/utils: Use download response URL to extract filename
    
    requests follows URL redirection by default for GET requests so
    update input URL to response one to ensure correct filename will
    be extracted from it.

    See https://jenkins.softwareheritage.org/job/DLDBASE/job/tests-on-diff/558/ for more details.

  • Antoine Lambert
    Antoine Lambert @anlambert ·
    Author Maintainer

    Merge request was merged

  • Antoine Lambert closed

    closed

Please register or sign in to reply