Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

build and smoke checkers #11

Merged
merged 2 commits into from
Nov 8, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
22 changes: 19 additions & 3 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,8 @@ The detailed result of the checks are described in a [JSON schema](https://grafa
- `license` - checks whether there is a suitable OSS license
- `git` - checks if the directory is git workdir
- `versions` - checks for semantic versioning git tags
- `build` - checks if k6 can be built with the extension
- `smoke` - checks if the smoke test script exists and runs successfully

## Install

Expand Down Expand Up @@ -58,7 +60,11 @@ Details
✔ git
found git worktree
✔ versions
found `6` versions, the latest is `v0.3.0`
found `13` versions, the latest is `v1.0.0`
✔ build
can be built with the latest k6 version
✔ smoke
`smoke.test.ts` successfully run with k6

```

Expand Down Expand Up @@ -99,14 +105,24 @@ Details
"passed": true
},
{
"details": "found `6` versions, the latest is `v0.3.0`",
"details": "found `13` versions, the latest is `v1.0.0`",
"id": "versions",
"passed": true
},
{
"details": "can be built with the latest k6 version",
"id": "build",
"passed": true
},
{
"details": "`smoke.test.ts` successfully run with k6",
"id": "smoke",
"passed": true
}
],
"grade": "A",
"level": 100,
"timestamp": 1724833956
"timestamp": 1731058317
}
```
</details>
Expand Down
60 changes: 55 additions & 5 deletions checker_module.go
Original file line number Diff line number Diff line change
Expand Up @@ -2,14 +2,18 @@ package k6lint

import (
"context"
"fmt"
"os"
"os/exec"
"path/filepath"
"regexp"

"golang.org/x/mod/modfile"
)

type moduleChecker struct {
file *modfile.File
exe string
}

func newModuleChecker() *moduleChecker {
Expand All @@ -32,12 +36,9 @@ func (mc *moduleChecker) hasGoModule(_ context.Context, dir string) *checkResult
return checkPassed("found `%s` as go module", mc.file.Module.Mod.String())
}

func (mc *moduleChecker) hasNoReplace(ctx context.Context, dir string) *checkResult {
func (mc *moduleChecker) hasNoReplace(_ context.Context, _ string) *checkResult {
if mc.file == nil {
res := mc.hasGoModule(ctx, dir)
if !res.passed {
return res
}
return checkFailed("missing go.mod")
}

if len(mc.file.Replace) != 0 {
Expand All @@ -46,3 +47,52 @@ func (mc *moduleChecker) hasNoReplace(ctx context.Context, dir string) *checkRes

return checkPassed("no `replace` directive in the `go.mod` file")
}

func (mc *moduleChecker) canBuild(ctx context.Context, dir string) *checkResult {
if mc.file == nil {
return checkFailed("missing go.mod")
}

exe, err := build(ctx, mc.file.Module.Mod.Path, dir)
if err != nil {
return checkError(err)
}

mc.exe = exe

return checkPassed("can be built with the latest k6 version")
}

var reSmoke = regexp.MustCompile(`(?i)^smoke(\.test)?\.(?:js|ts)`)

//nolint:forbidigo
func (mc *moduleChecker) smoke(ctx context.Context, dir string) *checkResult {
if mc.exe == "" {
return checkFailed("can't build")
}

filename, sortname, err := findFile(reSmoke,
dir,
filepath.Join(dir, "test"),
filepath.Join(dir, "tests"),
filepath.Join(dir, "examples"),
)
if err != nil {
return checkError(err)
}

if len(sortname) == 0 {
return checkFailed("no smoke test file found")
}

cmd := exec.CommandContext(ctx, mc.exe, "run", "--no-usage-report", "--no-summary", "--quiet", filename) //nolint:gosec

out, err := cmd.CombinedOutput()
if err != nil {
fmt.Fprintln(os.Stderr, string(out))

return checkError(err)
}

return checkPassed("`%s` successfully run with k6", sortname)
}
11 changes: 4 additions & 7 deletions checker_readme.go
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@ package k6lint

import (
"context"
"os"
"regexp"
)

Expand All @@ -11,15 +10,13 @@ var reREADME = regexp.MustCompile(
)

func checkerReadme(_ context.Context, dir string) *checkResult {
entries, err := os.ReadDir(dir) //nolint:forbidigo
_, name, err := findFile(reREADME, dir)
if err != nil {
return checkFailed("")
return checkError(err)
}

for _, entry := range entries {
if reREADME.Match([]byte(entry.Name())) {
return checkPassed("found `%s` as README file", entry.Name())
}
if len(name) > 0 {
return checkPassed("found `%s` as README file", name)
}

return checkFailed("no README file found")
Expand Down
102 changes: 102 additions & 0 deletions checker_util.go
Original file line number Diff line number Diff line change
@@ -0,0 +1,102 @@
package k6lint

import (
"bytes"
"context"
"fmt"
"io"
"log/slog"
"os"
"path/filepath"
"regexp"
"runtime"

"github.com/grafana/k6foundry"
)

//nolint:forbidigo
func build(ctx context.Context, module string, dir string) (filename string, result error) {
exe, err := os.CreateTemp("", "k6-*.exe")
if err != nil {
return "", err
}

if err = os.Chmod(exe.Name(), 0o700); err != nil { //nolint:gosec
return "", err
}

var out bytes.Buffer

defer func() {
if result != nil {
_, _ = io.Copy(os.Stderr, &out)
fmt.Fprintln(os.Stderr)
}
}()

builder, err := k6foundry.NewNativeBuilder(
ctx,
k6foundry.NativeBuilderOpts{
Logger: slog.New(slog.NewTextHandler(&out, &slog.HandlerOptions{Level: slog.LevelError})),
Stdout: &out,
Stderr: &out,
GoOpts: k6foundry.GoOpts{
CopyGoEnv: true,
Env: map[string]string{"GOWORK": "off"},
},
},
)
if err != nil {
result = err
return "", result
}

_, result = builder.Build(
ctx,
k6foundry.NewPlatform(runtime.GOOS, runtime.GOARCH),
"latest",
[]k6foundry.Module{{Path: module, ReplacePath: dir}},
nil,
exe,
)

if result != nil {
return "", result
}

if err = exe.Close(); err != nil {
return "", err
}

return exe.Name(), nil
}

//nolint:forbidigo
func findFile(rex *regexp.Regexp, dirs ...string) (string, string, error) {
for idx, dir := range dirs {
entries, err := os.ReadDir(dir)
if err != nil {
if idx == 0 {
return "", "", err
}

continue
}

script := ""

for _, entry := range entries {
if rex.Match([]byte(entry.Name())) {
script = entry.Name()

break
}
}

if len(script) > 0 {
return filepath.Join(dir, script), script, nil
}
}

return "", "", nil
}
4 changes: 3 additions & 1 deletion checks.go
Original file line number Diff line number Diff line change
Expand Up @@ -39,10 +39,12 @@ func checkDefinitions() []checkDefinition {
{id: CheckerModule, score: 2, fn: modCheck.hasGoModule},
{id: CheckerReplace, score: 2, fn: modCheck.hasNoReplace},
{id: CheckerReadme, score: 5, fn: checkerReadme},
{id: CheckerExamples, score: 5, fn: checkerExamples},
{id: CheckerExamples, score: 2, fn: checkerExamples},
{id: CheckerLicense, score: 5, fn: checkerLicense},
{id: CheckerGit, score: 1, fn: gitCheck.isWorkDir},
{id: CheckerVersions, score: 5, fn: gitCheck.hasVersions},
{id: CheckerBuild, score: 5, fn: modCheck.canBuild},
{id: CheckerSmoke, score: 2, fn: modCheck.smoke},
}

return defs
Expand Down
2 changes: 2 additions & 0 deletions compliance_gen.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 3 additions & 1 deletion docs/compliance.schema.json
Original file line number Diff line number Diff line change
Expand Up @@ -81,7 +81,9 @@
"examples",
"license",
"git",
"versions"
"versions",
"build",
"smoke"
]
}
}
Expand Down
2 changes: 2 additions & 0 deletions docs/compliance.schema.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -77,3 +77,5 @@ $defs:
- license
- git
- versions
- build
- smoke
14 changes: 12 additions & 2 deletions docs/example.json
Original file line number Diff line number Diff line change
Expand Up @@ -31,12 +31,22 @@
"passed": true
},
{
"details": "found `6` versions, the latest is `v0.3.0`",
"details": "found `13` versions, the latest is `v1.0.0`",
"id": "versions",
"passed": true
},
{
"details": "can be built with the latest k6 version",
"id": "build",
"passed": true
},
{
"details": "`smoke.test.ts` successfully run with k6",
"id": "smoke",
"passed": true
}
],
"grade": "A",
"level": 100,
"timestamp": 1724833956
"timestamp": 1731058317
}
6 changes: 5 additions & 1 deletion docs/example.txt
Original file line number Diff line number Diff line change
Expand Up @@ -17,5 +17,9 @@ Details
✔ git
found git worktree
✔ versions
found `6` versions, the latest is `v0.3.0`
found `13` versions, the latest is `v1.0.0`
✔ build
can be built with the latest k6 version
✔ smoke
`smoke.test.ts` successfully run with k6

1 change: 1 addition & 0 deletions go.mod
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@ require (
github.com/go-enry/go-license-detector/v4 v4.3.1
github.com/go-git/go-git/v5 v5.12.0
github.com/grafana/clireadme v0.1.0
github.com/grafana/k6foundry v0.3.0
github.com/mattn/go-colorable v0.1.13
github.com/spf13/cobra v1.8.1
golang.org/x/mod v0.21.0
Expand Down
2 changes: 2 additions & 0 deletions go.sum
Original file line number Diff line number Diff line change
Expand Up @@ -56,6 +56,8 @@ github.com/google/go-cmp v0.6.0 h1:ofyhxvXcZhMsU5ulbFiLKl/XBFqE1GSq7atu8tAmTRI=
github.com/google/go-cmp v0.6.0/go.mod h1:17dUlkBOakJ0+DkrSSNjCkIjxS6bF9zb3elmeNGIjoY=
github.com/grafana/clireadme v0.1.0 h1:KYEYSnYdSzmHf3bufaK6fQZ5j4dzvM/T+G6Ba+qNnAM=
github.com/grafana/clireadme v0.1.0/go.mod h1:Wy4KIG2ZBGMYAYyF9l7qAy+yoJVasqk/txsRgoRI3gc=
github.com/grafana/k6foundry v0.3.0 h1:C+6dPbsOv7Uq4hEhBFNuYqmTdE9jQ0VqhXqBDtMkVTE=
github.com/grafana/k6foundry v0.3.0/go.mod h1:/NtBSQQgXup5SVbfInl0Q8zKVx08xgvXIZ0xncqexEs=
github.com/hhatto/gorst v0.0.0-20181029133204-ca9f730cac5b h1:Jdu2tbAxkRouSILp2EbposIb8h4gO+2QuZEn3d9sKAc=
github.com/hhatto/gorst v0.0.0-20181029133204-ca9f730cac5b/go.mod h1:HmaZGXHdSwQh1jnUlBGN2BeEYOHACLVGzYOXCbsLvxY=
github.com/inconshreveable/mousetrap v1.1.0 h1:wN+x4NVGpMsO7ErUn/mUI3vEoE6Jt13X2s0bqwp9tc8=
Expand Down
21 changes: 21 additions & 0 deletions releases/v0.2.0.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,21 @@
k6lint `v0.2.0` is here 🎉!

This version adds two new checkers to the linter:
- **build checker**
- **smoke checker**

## build checker

The check is successful if the extension can be built with the latest k6 release.

## smoke checker

The check is successful if there is a smoke test script and it runs successfully with the k6 built with the extension.

Obviously, a prerequisite for a successful run is that the build checker runs successfully, otherwise k6 cannot be built with the extension.

The smoke test script file is searched for in the root of the repository and in the `test`,`tests`,`examples` directories. The name of the smoke test script is one of the following:
- `smoke.js`
- `smoke.ts`
- `smoke.test.js`
- `smoke.test.ts`
Loading