Skip to content
This repository has been archived by the owner on Sep 9, 2020. It is now read-only.

[WIP] VerifyDepTree and DigestFromDirectory use godirwalk #1084

Closed
wants to merge 30 commits into from
Closed

[WIP] VerifyDepTree and DigestFromDirectory use godirwalk #1084

wants to merge 30 commits into from

Commits on Sep 5, 2017

  1. digest uses godirwalk

    @karrick
    karrick authored and Karrick S. McDermott committed Sep 5, 2017
    Configuration menu
    Copy the full SHA
    502d835 View commit details
    Browse the repository at this point in the history

  2. more simplified node type detection

    @karrick
    karrick authored and Karrick S. McDermott committed Sep 5, 2017
    Configuration menu
    Copy the full SHA
    f149f35 View commit details
    Browse the repository at this point in the history

  3. digestNoAction constant name describes purpose of values

    @karrick
    karrick authored and Karrick S. McDermott committed Sep 5, 2017
    Configuration menu
    Copy the full SHA
    6dd14f9 View commit details
    Browse the repository at this point in the history

  4. updated godirwalk

    @karrick
    karrick authored and Karrick S. McDermott committed Sep 5, 2017
    Configuration menu
    Copy the full SHA
    157c30f View commit details
    Browse the repository at this point in the history

  5. updated godirwalk

    @karrick
    karrick authored and Karrick S. McDermott committed Sep 5, 2017
    Configuration menu
    Copy the full SHA
    ab1cacf View commit details
    Browse the repository at this point in the history

  6. update godirwalk to cleaner version v1.2.4

    @karrick
    karrick authored and Karrick S. McDermott committed Sep 5, 2017
    Configuration menu
    Copy the full SHA
    4109fb9 View commit details
    Browse the repository at this point in the history

  7. eliminated unused variables 

    * found by megacheck :)
    @karrick
    karrick authored and Karrick S. McDermott committed Sep 5, 2017
    Configuration menu
    Copy the full SHA
    be2c05c View commit details
    Browse the repository at this point in the history

  8. vendor'd package in accordance with guidance from travis ci

    @karrick
    karrick authored and Karrick S. McDermott committed Sep 5, 2017
    Configuration menu
    Copy the full SHA
    ac6f562 View commit details
    Browse the repository at this point in the history

Commits on Sep 10, 2017

  1. Don't assume every git repository has a HEAD 

    Fixes golang#1051
    @BartSchuurmans
    BartSchuurmans authored and Karrick S. McDermott committed Sep 10, 2017
    Configuration menu
    Copy the full SHA
    e058b2f View commit details
    Browse the repository at this point in the history

  2. Add test for gitSource.listVersions() without HEAD

    @BartSchuurmans
    BartSchuurmans authored and Karrick S. McDermott committed Sep 10, 2017
    Configuration menu
    Copy the full SHA
    8a131bd View commit details
    Browse the repository at this point in the history

  3. cmd/dep: Standardize import logic for importers 

    The importers are now only responsible for reading config files and
feeding unvalidated lock and constraint hints into importPackages.
    @carolynvs
    carolynvs authored and Karrick S. McDermott committed Sep 10, 2017
    Configuration menu
    Copy the full SHA
    caca12a View commit details
    Browse the repository at this point in the history

  4. cmd/dep: Support new importer rules 

    * Don’t throw away a locked version match due to a conflicting
  constraint unless there is another match that _does_ match
* Ignore pinned constraints
* Ignore conflicting constraints
    @carolynvs
    carolynvs authored and Karrick S. McDermott committed Sep 10, 2017
    Configuration menu
    Copy the full SHA
    41d353e View commit details
    Browse the repository at this point in the history

  5. cmd/dep: Switch to special importer testdata set 

    github.com/carolynvs/deptest-importers contains the entire matrix of
testdata needed by the importers
    @carolynvs
    carolynvs authored and Karrick S. McDermott committed Sep 10, 2017
    Configuration menu
    Copy the full SHA
    d61e44e View commit details
    Browse the repository at this point in the history

  6. cmd/dep: Move import test execution into testcase 

    * The setup and validation for all the importer testcases are the same,
  they just need to handle calling the importer properly.
* Flag importer tests to run in parallel now that they've been cleaned up.
    @carolynvs
    carolynvs authored and Karrick S. McDermott committed Sep 10, 2017
    Configuration menu
    Copy the full SHA
    fa8e165 View commit details
    Browse the repository at this point in the history

  7. cmd/dep: Validate warnings in import testcases

    @carolynvs
    carolynvs authored and Karrick S. McDermott committed Sep 10, 2017
    Configuration menu
    Copy the full SHA
    acd473f View commit details
    Browse the repository at this point in the history

  8. cmd/dep: Document every baseImporter function

    @carolynvs
    carolynvs authored and Karrick S. McDermott committed Sep 10, 2017
    Configuration menu
    Copy the full SHA
    f0e0e3a View commit details
    Browse the repository at this point in the history

  9. cmd/dep: Simplify pinned constraint check

    @carolynvs
    carolynvs authored and Karrick S. McDermott committed Sep 10, 2017
    Configuration menu
    Copy the full SHA
    7d5628c View commit details
    Browse the repository at this point in the history

  10. cmd/dep: fix parallel tests 

    The closure created by t.Run wasn't copying the testcase value, so the
test was being run on the wrong values.
    @carolynvs
    carolynvs authored and Karrick S. McDermott committed Sep 10, 2017
    Configuration menu
    Copy the full SHA
    7e85ef6 View commit details
    Browse the repository at this point in the history

  11. cmd/dep: verify empty locks are skipped 

    When a lock hint is empty, nothing should be added to the lock
    @carolynvs
    carolynvs authored and Karrick S. McDermott committed Sep 10, 2017
    Configuration menu
    Copy the full SHA
    49a2eb4 View commit details
    Browse the repository at this point in the history

  12. gps: source cache: fix flaky bolt test clock by forcing future timestamp

    @jmank88
    jmank88 authored and Karrick S. McDermott committed Sep 10, 2017
    Configuration menu
    Copy the full SHA
    0321346 View commit details
    Browse the repository at this point in the history

  13. Fix naming convention for test

    @BartSchuurmans
    BartSchuurmans authored and Karrick S. McDermott committed Sep 10, 2017
    Configuration menu
    Copy the full SHA
    94dec0f View commit details
    Browse the repository at this point in the history

  14. Nicer error reporting in test

    @BartSchuurmans
    BartSchuurmans authored and Karrick S. McDermott committed Sep 10, 2017
    Configuration menu
    Copy the full SHA
    59d1d75 View commit details
    Browse the repository at this point in the history

  15. Don't rely on local git config in test 

    Fixes an error when the git user is not set.
    @BartSchuurmans
    BartSchuurmans authored and Karrick S. McDermott committed Sep 10, 2017
    Configuration menu
    Copy the full SHA
    5332f3a View commit details
    Browse the repository at this point in the history

  16. Fix test when git is not set up

    @BartSchuurmans
    BartSchuurmans authored and Karrick S. McDermott committed Sep 10, 2017
    Configuration menu
    Copy the full SHA
    f380ff7 View commit details
    Browse the repository at this point in the history

  17. Fix file:// URI on Windows

    @BartSchuurmans
    BartSchuurmans authored and Karrick S. McDermott committed Sep 10, 2017
    Configuration menu
    Copy the full SHA
    5c9e128 View commit details
    Browse the repository at this point in the history

  18. gps: fix unwrapVcsErr to handle nil causes

    @jmank88
    jmank88 authored and Karrick S. McDermott committed Sep 10, 2017
    Configuration menu
    Copy the full SHA
    3d84357 View commit details
    Browse the repository at this point in the history

  19. gps: add unwrapVcsErr nil cause test

    @jmank88
    jmank88 authored and Karrick S. McDermott committed Sep 10, 2017
    Configuration menu
    Copy the full SHA
    4626497 View commit details
    Browse the repository at this point in the history

  20. cmd/dep: fix setting importer tests as parallel 

    Use the test helper so that it can check for any problems up-front.
    @carolynvs
    carolynvs authored and Karrick S. McDermott committed Sep 10, 2017
    Configuration menu
    Copy the full SHA
    5e17c34 View commit details
    Browse the repository at this point in the history

  21. cmd/dep: disable running importer tests in parallel 

    Disable until we can figure out why this is causing the
following error on the AppVeyor(Windows) builds:

"remote repository at https://github.com/carolynvs/deptest-importers does
 not exist, or is inaccessible"
    @carolynvs
    carolynvs authored and Karrick S. McDermott committed Sep 10, 2017
    Configuration menu
    Copy the full SHA
    a1f23c2 View commit details
    Browse the repository at this point in the history

  22. Clarify when to run ensure if vendor is not committed

    @carolynvs
    carolynvs authored and Karrick S. McDermott committed Sep 10, 2017
    Configuration menu
    Copy the full SHA
    b40a93c View commit details
    Browse the repository at this point in the history