search

mbj4668 / pyang

An extensible YANG validator and converter in python
ISC License
530 stars 342 forks source link

"make check" / "make test" #837

Open mcr opened 1 year ago

mcr commented 1 year ago

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 commented 1 year ago

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 commented 1 year ago

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'
mbj4668 commented 1 year ago

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