actions/gotesttools-gotestfmt-action
3
0
Fork 0
mirror of https://github.com/GoTestTools/gotestfmt-action.git synced 2026-02-08 08:47:28 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions
45 commits 3 branches 4 tags 834 KiB
Commit graph

12 commits

Author SHA1 Message Date
Y. Meyer-Norwood
02b936e80b Fix installation on Windows runners (#10) 
* Correctly identify windows os

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Prevent darwin from being reported as Linux

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Remove unix specific logging and determine .zip extension on windows

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Add selftest on windows

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Run self test on all push events (for testing)

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Inline postfix declaration

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Separate extension detection from postfix

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Correctly avoid assignment to const variable

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Make postfix constant

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Prevent curl from assuming tar.gz format

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Refactor download stage

* Separate logical steps
* Use string interpolation for curl command

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Use environment defined temp path

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Fix missing `$`

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Correct all references to hardcoded filename

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Attempt debugging missing download

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Restore original console loggers

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* USe GOOs

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Extract binary directly to GOPATH rather than hardcoded lib path

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Properly log GOOS detection

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Remove GOOS detection as GOOS not in environment

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Replace GOPATH with system dependent lib path

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Fix windows path separator problems

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Simplify archive path

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Fix escape character

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Use more meaningful name for binpath

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Fix whitespace

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Add debugger lines

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Add debugger lines for executed commands

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Use console debug instead of log

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Use program files as binpath

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Add install directory to path

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Remove debuggers

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Fix os check

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Remove redundant pathsep variable

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Add documentation comments

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Fix missing path separators

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Explicitly echo bin path into GITHUB_PATH

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Quote bin path when `cd`ing into it to avoid shell globbing

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Ensure bin path exists

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Attempt to prevent existing directories causing problems

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Use core.addPath to add binpath to job's PATH

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Attempt to use pre-existing lib path instead of creating new one

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Add debugger for PATH variable

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* List System32 files to look for gotestfmt binary

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Remove debugger

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Manually setup binpath and add it to system path

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* List PATH targets for debugging

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Add debuggers for userprofile

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* List install target's contents

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* List files recursively and verbosely

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Search `/bin` specifically

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Only manually add the binpath on windows

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* List all PATHs

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Split on correct separator

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Don't force the directory to exist/into PATH

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Attempt to create custom binpath in windows

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Fix mkdir invocation

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Create all path components

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* List all files in target directory

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Try using pwsh for Windows runner

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Try using exe for pwsh

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Remove testing step

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* List files extracted to bin path

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Expand debugger

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* List files in temporary directory

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* List files in current directory

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Rather than changing directories, specify target to tar command

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Quote parameters to tar command

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Specify target directory before target files

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Move extracted binary manually

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Add callback to file move

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Use system defined target file

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Correct interpolation

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Copy binfile instead of renaming to prevent cross-device linking

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* List installed files

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Use binfile in final success log message

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Remove explicit path separator

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Specify fs.constants.COPYFILE_EXCL copy mode

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Extract archive directly into workspace

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Remove redundant `binfile` variable

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Remove debugger

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* List all files in workspace

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Remove debugging steps

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Remove inline comments

This is per @engelmi's comment: https://github.com/GoTestTools/gotestfmt-action/pull/10/files#r1017614705

Signed-off-by: Y. Meyer-Norwood <106889957+norwd@users.noreply.github.com>

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>
Signed-off-by: Y. Meyer-Norwood <106889957+norwd@users.noreply.github.com>
 2022-12-02 18:57:25 +01:00
Yuri Norwood
07a236ddd6 Extend selftest workflow to multiple runners (#9) 
* Extend selftest workflow to multiple runners

This should help catch errors like #4 and #8 by ensuring that gotestfmt can run on all the major OS runners for GitHub Actions.

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Separate windows step from *nix runners

This allows `bash` and `pwsh` syntax to be used separately to avoid having to write the test as a polyglot 😬

Also added `secrets.GITHUB_TOKEN` as a fallback in case the repo secret `secrets.GH_TOKEN` is not set up, e.g. in a fork.

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Remove `/tmp` directory for artifacts

On MacOS, this results in:

```
Error: Provided rootDirectory /tmp is not a valid directory
```

On Windows this results in:

```
Error: No files were found with the provided path: /tmp/gotest.windows.log. No artifacts will be uploaded.
```

Both seem to be unhappy with `/tmp`, since the workspace is implicitly cleaned up at the end of the workflow, there shouldn't be a need to specify `/tmp`.

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Name each workflow run of the matrix

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Fix artifact path

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Simplify workflow by just using bash universally

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

* Apply suggestions from code review

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>

Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com>
 2022-12-02 18:57:25 +01:00
Michael Engel
59eb7e3506 added token to install (#7) 
* added token to install
* removed on push trigger for selftest
 2022-12-02 18:57:25 +01:00
Michael Engel
bddb983f7c added selftest for macos 2022-12-02 18:57:25 +01:00
Janos Bonic
3e91a15d55
Issue haveyoudebuggedit/gotestfmt#8: JSON support (#1) 2021-12-05 12:54:21 +00:00
Janos
ec3a52825f
More testing 2021-07-28 09:21:31 +02:00
Janos
03ad8ff09e
More testing 2021-07-28 09:19:39 +02:00
Janos
c3cb9a3d0c
More testing 2021-07-28 09:19:07 +02:00
Janos
fc8db34b00
More testing 2021-07-28 09:18:28 +02:00
Janos
1f6356c0db
More testing 2021-07-28 09:07:26 +02:00
Janos
2a45d3ea84
More testing 2021-07-28 08:30:44 +02:00
Janos
8a260b16ec
Testing actions 2021-07-28 08:24:31 +02:00