pyang icon indicating copy to clipboard operation
pyang copied to clipboard

"make check" / "make test"

Open mcr opened this issue 2 years ago • 3 comments

I see that "make test" runs lint, which uses flake8 to verify style. Are there any functional/unit test cases that I'm overlooking?

mcr avatar Jan 10 '23 17:01 mcr

Like, why doesn't it, "make -C test test" ? (And why didn't the (cd test && make test) that I now notice not work for me the first time)

mcr avatar Jan 10 '23 17:01 mcr

I can't get "make test" to work, test_verbose is really backwards done, and I seem to randomly get:

/sandel/src/pyang/test/../bin/pyang --mef mef-yt10.yang || exit 1; \
        /sandel/src/pyang/test/../bin/pyang --mef mef-yt11.yang || exit 1; \
        /sandel/src/pyang/test/../bin/pyang --ieee ieee-yt12.yang || exit 1;
mef-yt10.yang:899: warning: line length 84 exceeds 70 characters
mef-yt11.yang:899: warning: line length 84 exceeds 70 characters
checking yt2.yang.gen.yang...error yt2.yang.gen.yang: [Errno 2] No such file or directory: 'yt2.yang.gen.yang'

mcr avatar Jan 11 '23 18:01 mcr

About the random errors, this can be resolved by a make clean. The problem is fixed in 902c578.

mbj4668 avatar Jan 13 '23 08:01 mbj4668