| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168 |
- package project
- import (
- "os"
- "path/filepath"
- "strings"
- "testing"
- "git.monoexp.com/bajdev/mpckit/internal/xpj"
- )
- // loadChanged reads the device's DemoFile, applies change to its data, writes
- // it to a temporary folder and loads it.
- func loadChanged(t *testing.T, change func(d *xpj.Node)) error {
- t.Helper()
- f, err := xpj.ReadFile(filepath.Join(testdata, "device", "DemoFile.xpj"))
- must(t, err)
- d, err := f.Data()
- must(t, err)
- change(d)
- path := filepath.Join(t.TempDir(), "Changed.xpj")
- must(t, f.WriteFile(path))
- _, _, err = Load(path)
- return err
- }
- func set(t *testing.T, d *xpj.Node, path string, v *xpj.Node) {
- t.Helper()
- must(t, d.Set(path, v))
- }
- func remove(t *testing.T, d *xpj.Node, path, key string) {
- t.Helper()
- n, err := d.Get(path)
- must(t, err)
- if !n.DeleteKey(key) {
- t.Fatalf("%s has no key %q", path, key)
- }
- }
- func TestLoadRejectsMalformed(t *testing.T) {
- for _, tc := range []struct {
- name string
- change func(t *testing.T, d *xpj.Node)
- want string // in the error
- }{
- {"parent folder", func(t *testing.T, d *xpj.Node) {
- set(t, d, "samples[0].path", xpj.NewString("../x.wav"))
- }, `samples[0].path: file name "../x.wav" contains a folder separator`},
- {"subfolder", func(t *testing.T, d *xpj.Node) {
- set(t, d, "samples[1].path", xpj.NewString("a/b.wav"))
- }, `samples[1].path: file name "a/b.wav" contains a folder separator`},
- {"backslash", func(t *testing.T, d *xpj.Node) {
- set(t, d, "samples[1].path", xpj.NewString(`..\b.wav`))
- }, `samples[1].path: file name "..\\b.wav" contains a folder separator`},
- {"dot dot", func(t *testing.T, d *xpj.Node) {
- set(t, d, "samples[2].path", xpj.NewString(".."))
- }, `samples[2].path: ".." is not a file name`},
- {"empty", func(t *testing.T, d *xpj.Node) {
- set(t, d, "samples[0].path", xpj.NewString(""))
- }, `samples[0].path: file name is empty`},
- {"control character", func(t *testing.T, d *xpj.Node) {
- set(t, d, "samples[0].path", xpj.NewString("a\x00.wav"))
- }, `samples[0].path: file name "a\x00.wav" contains a control character`},
- {"track pool", func(t *testing.T, d *xpj.Node) {
- set(t, d, "tracks[0].samples[0].path", xpj.NewString("../x.wav"))
- }, `tracks[0].samples[0].path: file name "../x.wav"`},
- {"missing path", func(t *testing.T, d *xpj.Node) {
- remove(t, d, "samples[3]", "path")
- }, "samples[3].path is missing or not text"},
- {"path not text", func(t *testing.T, d *xpj.Node) {
- set(t, d, "samples[3].path", xpj.NewInt(3))
- }, "samples[3].path is missing or not text"},
- {"layer outside", func(t *testing.T, d *xpj.Node) {
- set(t, d, "tracks[0].program.drum.instruments[17].layersv[2].sampleFile", xpj.NewString("../x.wav"))
- }, `pad B2 layer 3 sampleFile: file name "../x.wav" contains a folder separator`},
- {"layer not text", func(t *testing.T, d *xpj.Node) {
- set(t, d, "tracks[0].program.drum.instruments[0].layersv[0].sampleFile", xpj.NewNull())
- }, "pad A1 layer 1 sampleFile is not text"},
- {"missing bpm", func(t *testing.T, d *xpj.Node) {
- remove(t, d, "sequences[0].value", "bpm")
- }, "sequences[0]: bpm is missing or not a number"},
- {"missing key", func(t *testing.T, d *xpj.Node) {
- remove(t, d, "sequences[1]", "key")
- }, "sequences[1]: key is missing or not a whole number"},
- {"missing value", func(t *testing.T, d *xpj.Node) {
- set(t, d, "sequences[1].value", xpj.NewArray())
- }, "sequences[1]: value is missing or not an object"},
- {"missing length", func(t *testing.T, d *xpj.Node) {
- remove(t, d, "sequences[0].value", "lengthBars")
- }, "sequences[0]: lengthBars is missing or not a whole number"},
- {"name not text", func(t *testing.T, d *xpj.Node) {
- set(t, d, "sequences[0].value.name", xpj.NewInt(1))
- }, "sequences[0]: name is missing or not text"},
- } {
- t.Run(tc.name, func(t *testing.T) {
- err := loadChanged(t, func(d *xpj.Node) { tc.change(t, d) })
- if err == nil || !strings.Contains(err.Error(), tc.want) {
- t.Fatalf("got error %v, want one containing %q", err, tc.want)
- }
- })
- }
- }
- func TestLoadAcceptsMissingLayerFile(t *testing.T) {
- err := loadChanged(t, func(d *xpj.Node) {
- remove(t, d, "tracks[0].program.drum.instruments[0].layersv[0]", "sampleFile")
- })
- must(t, err)
- }
- func TestLoadMissingVersion(t *testing.T) {
- f, err := xpj.ReadFile(filepath.Join(testdata, "device", "DemoFile.xpj"))
- must(t, err)
- d, err := f.Data()
- must(t, err)
- d.DeleteKey("version")
- path := filepath.Join(t.TempDir(), "NoVersion.xpj")
- must(t, f.WriteFile(path))
- _, warnings, err := Load(path)
- must(t, err)
- if len(warnings) != 1 || !strings.Contains(warnings[0], "version is missing") {
- t.Fatalf("warnings = %q", warnings)
- }
- }
- func TestPlainFileName(t *testing.T) {
- for _, ok := range []string{"kick.wav", "80s-Kik-Dreamer2 SA.wav", "a..b.wav", ".hidden.wav", "Ünïcode.wav"} {
- if err := PlainFileName(ok); err != nil {
- t.Errorf("PlainFileName(%q) = %v", ok, err)
- }
- }
- for _, bad := range []string{"", ".", "..", "../x.wav", "a/b.wav", `a\b.wav`, "/x.wav", "x\n.wav", "x\x7f.wav"} {
- if err := PlainFileName(bad); err == nil {
- t.Errorf("PlainFileName(%q) = nil", bad)
- }
- }
- }
- // FuzzLoad loads arbitrary files; none may make the read-only calls panic.
- func FuzzLoad(f *testing.F) {
- seeds, err := filepath.Glob(filepath.Join(testdata, "device", "*.xpj"))
- if err != nil || len(seeds) == 0 {
- f.Fatalf("no seed files: %v", err)
- }
- for _, s := range seeds {
- b, err := os.ReadFile(s)
- if err != nil {
- f.Fatal(err)
- }
- f.Add(b)
- }
- f.Fuzz(func(t *testing.T, b []byte) {
- path := filepath.Join(t.TempDir(), "Fuzz.xpj")
- if err := os.WriteFile(path, b, 0o644); err != nil {
- t.Fatal(err)
- }
- p, _, err := Load(path)
- if err != nil {
- return
- }
- for slot := range Pads {
- _, _ = p.Pad(slot)
- }
- p.Sequences()
- p.PoolFiles()
- })
- }
|