Closed mpawlowski closed 1 year ago
This PR should fix the issues mentioned in https://github.com/rumblefrog/go-a2s/issues/8. I also made some additional changes outlined below.
test-host
go test
➜ marcin@hive go-a2s git:(master) go test -run FuzzParsePlayerInfo -fuzz FuzzParsePlayerInfo -fuzztime 1m fuzz: elapsed: 0s, gathering baseline coverage: 0/232 completed fuzz: elapsed: 0s, gathering baseline coverage: 232/232 completed, now fuzzing with 24 workers fuzz: elapsed: 3s, execs: 631703 (210548/sec), new interesting: 0 (total: 232) fuzz: elapsed: 6s, execs: 1390831 (252978/sec), new interesting: 0 (total: 232) fuzz: elapsed: 9s, execs: 2166189 (258514/sec), new interesting: 0 (total: 232) fuzz: elapsed: 12s, execs: 2939420 (257733/sec), new interesting: 0 (total: 232) fuzz: elapsed: 15s, execs: 3706793 (255822/sec), new interesting: 0 (total: 232) fuzz: elapsed: 18s, execs: 4467026 (253365/sec), new interesting: 0 (total: 232) fuzz: elapsed: 21s, execs: 5231794 (254942/sec), new interesting: 0 (total: 232) fuzz: elapsed: 24s, execs: 6001452 (256567/sec), new interesting: 0 (total: 232) fuzz: elapsed: 27s, execs: 6769540 (256026/sec), new interesting: 0 (total: 232) fuzz: elapsed: 30s, execs: 7530532 (253620/sec), new interesting: 0 (total: 232) fuzz: elapsed: 33s, execs: 8304380 (258015/sec), new interesting: 0 (total: 232) fuzz: elapsed: 36s, execs: 9066076 (253856/sec), new interesting: 0 (total: 232) fuzz: elapsed: 39s, execs: 9842515 (258786/sec), new interesting: 0 (total: 232) fuzz: elapsed: 42s, execs: 10605470 (254377/sec), new interesting: 0 (total: 232) fuzz: elapsed: 45s, execs: 11364872 (253105/sec), new interesting: 0 (total: 232) fuzz: elapsed: 48s, execs: 12133514 (256235/sec), new interesting: 0 (total: 232) fuzz: elapsed: 51s, execs: 12899618 (255320/sec), new interesting: 0 (total: 232) fuzz: elapsed: 54s, execs: 13654329 (251574/sec), new interesting: 0 (total: 232) fuzz: elapsed: 57s, execs: 14413669 (253142/sec), new interesting: 0 (total: 232) fuzz: elapsed: 1m0s, execs: 15180401 (255467/sec), new interesting: 0 (total: 232) fuzz: elapsed: 1m0s, execs: 15180401 (0/sec), new interesting: 0 (total: 232) PASS ok github.com/rumblefrog/go-a2s 60.116s
commit 6ba9f84466abd24286e787383de34ec50fe8a4b5 Author: Marcin Pawlowski github@marcin.io Date: Mon Dec 12 16:13:14 2022 -0800
cleanup some errors
commit 49bc058b27911639ce154267f32e087f92a8adc8 Author: Marcin Pawlowski github@marcin.io Date: Mon Dec 12 16:07:53 2022 -0800
clean up errors
commit ba93af67a2e2e8c6e660be3077f18f2837fb6335 Author: Marcin Pawlowski github@marcin.io Date: Mon Dec 12 15:43:26 2022 -0800
fix player-info panics
Thanks for this!
This PR should fix the issues mentioned in https://github.com/rumblefrog/go-a2s/issues/8. I also made some additional changes outlined below.
Additonal Changes
test-host
.go test
.Commits
commit 6ba9f84466abd24286e787383de34ec50fe8a4b5 Author: Marcin Pawlowski github@marcin.io Date: Mon Dec 12 16:13:14 2022 -0800
commit 49bc058b27911639ce154267f32e087f92a8adc8 Author: Marcin Pawlowski github@marcin.io Date: Mon Dec 12 16:07:53 2022 -0800
commit ba93af67a2e2e8c6e660be3077f18f2837fb6335 Author: Marcin Pawlowski github@marcin.io Date: Mon Dec 12 15:43:26 2022 -0800