Skip to content

x/tools/refactor/rename: TestMoves is failing on Windows #16384

Closed
@bradfitz

Description

@bradfitz
--- FAIL: TestMoves (0.01s)
    mvpkg_test.go:403: -from "b" -to "x/a": unexpected warnings:
        \go\src\conflict\0.go:4:8: renaming this imported package name "b" to "a"
        \go\src\conflict\0.go:3:8:  conflicts with imported package name in same block
        \go\src\conflict\0.go:3:8: skipping update of this file
        want:
        /go/src/conflict/0.go:4:8: renaming this imported package name "b" to "a"
        /go/src/conflict/0.go:3:8:  conflicts with imported package name in same block
        /go/src/conflict/0.go:3:8: skipping update of this file
    mvpkg_test.go:418: -from "b" -to "x/a": rewritten file /go/src/ok/0.go does not match expectation; got <<<package ok

        import a "x\\a"

        var _ a.B
        >>>
        want <<<package ok

        import "x/a"

        var _ a.B
        >>>
    mvpkg_test.go:418: -from "b" -to "x/a": rewritten file /go/src/conflict/0.go does not match expectation; got <<<package conflict

        import "a"
        import a "x\\a"

        var _ a.A
        var _ b.B
        >>>
        want <<<package conflict

        import "a"
        import "x/a"

        var _ a.A
        var _ b.B
        >>>
    mvpkg_test.go:395: -from "x/foo" -to "y/foo": unexpected error: invalid move destination: y/foo; package or subpackage x/foo already exists
Renamed 1 occurrence in 1 file in 1 package.
Renamed 2 occurrences in 1 file in 1 package.
Renamed 2 occurrences in 1 file in 1 package.
Renamed 1 occurrence in 1 file in 1 package.
Renamed 2 occurrences in 1 file in 1 package.
Renamed 1 occurrence in 1 file in 1 package.
Renamed 1 occurrence in 1 file in 1 package.
Renamed 3 occurrences in 1 file in 1 package.
Renamed 2 occurrences in 1 file in 1 package.
Renamed 2 occurrences in 1 file in 1 package.
Renamed 2 occurrences in 1 file in 1 package.
Renamed 2 occurrences in 2 files in 2 packages.
Renamed 2 occurrences in 1 file in 1 package.
Renamed 2 occurrences in 1 file in 1 package.
Renamed 4 occurrences in 1 file in 1 package.
Renamed 4 occurrences in 1 file in 1 package.
Renamed 4 occurrences in 1 file in 1 package.
Renamed 2 occurrences in 1 file in 1 package.
Renamed 2 occurrences in 1 file in 1 package.
Renamed 2 occurrences in 1 file in 1 package.
Renamed 2 occurrences in 1 file in 1 package.
Renamed 2 occurrences in 1 file in 1 package.
Renamed 3 occurrences in 2 files in 2 packages.
Renamed 3 occurrences in 2 files in 2 packages.
Renamed 3 occurrences in 2 files in 2 packages.
Renamed 5 occurrences in 1 file in 1 package.
Renamed 5 occurrences in 1 file in 1 package.
Renamed 3 occurrences in 1 file in 1 package.
Renamed 2 occurrences in 1 file in 1 package.
Renamed 1 occurrence in 1 file in 1 package.
Renamed 2 occurrences in 1 file in 1 package.
Renamed 1 occurrence in 1 file in 1 package.
Renamed 2 occurrences in 1 file in 1 package.
FAIL
FAIL    golang.org/x/tools/refactor/rename  0.213s

/cc @quentinmit (feel free to file bugs like this for all subrepo failures if none exist)

Activity

added
TestingAn issue that has been verified to require only test changes, not just a test failure.
on Jul 15, 2016
added this to the Unreleased milestone on Jul 15, 2016
gopherbot

gopherbot commented on Jul 15, 2016

@gopherbot
Contributor

CL https://golang.org/cl/24941 mentions this issue.

adonovan

adonovan commented on Jul 15, 2016

@adonovan
Member

Looks like two separate issues, both related to forward vs. backslashes as directory separators:

  • the import declaration uses backslashes (a bug in the package)
  • the actual warnings in the test use backslashes (a bug in the test)
gopherbot

gopherbot commented on Jul 18, 2016

@gopherbot
Contributor

CL https://golang.org/cl/24969 mentions this issue.

gopherbot

gopherbot commented on Oct 31, 2016

@gopherbot
Contributor

CL https://golang.org/cl/32392 mentions this issue.

locked and limited conversation to collaborators on Oct 31, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    FrozenDueToAgeOS-WindowsTestingAn issue that has been verified to require only test changes, not just a test failure.help wanted

    Type

    No type

    Projects

    No projects

    Relationships

    None yet

      Development

      No branches or pull requests

        Participants

        @bradfitz@adonovan@alandonovan@gopherbot

        Issue actions

          x/tools/refactor/rename: TestMoves is failing on Windows · Issue #16384 · golang/go