xeluxee / competitest.nvim

CompetiTest.nvim is a Neovim plugin for Competitive Programming: it can manage and check testcases, download problems and contests from online judges and much more
GNU Lesser General Public License v3.0
389 stars 16 forks source link

Tests are not run when a dot `.` is in the file name #4

Closed MuhammadSawalhy closed 2 years ago

MuhammadSawalhy commented 2 years ago

I guess that the problem that the code is only compiled with no test cases detected is due to the presence of a dot in the file name, A.Vanya-and-Fence.cpp. But when I changed the name to A.Vanya-and-Fence/main.cpp, tests are run successfully ❤️.

Note that :CompetiConvert auto also says no test cases detect even that they are already there and generated using :CompetiRecieve

https://codeforces.com/contest/677/problem/A

xeluxee commented 2 years ago

Thanks for reporting this issue This is caused by lua string functions interpreting dots and dashes as pattern special characters, so I have to find a way to escape all the characters that could mess up lua patterns system. I'll try to fix it asap.

Anyway in the meantime you can set testcases_use_single_file to true, since this storage method isn't affected by this issue

MuhammadSawalhy commented 2 years ago

I think the priority goes for "submitting" the code feature first. But, I think this issue won't take that much time to worry about priority. I may work on the submission feature and create a PR for it. But that won't be before three weeks or maybe more from now as I have no free time meanwhile.

xeluxee commented 2 years ago

Before working on "submit" feature we have to choose the right tool to submit. And since this is only a bugfix it could take me a shorter time than implementing a new feature.

I won't be available too in a short time to work on implementing new features, anyway any help would be appreciated

xeluxee commented 2 years ago

@MuhammadSawalhy after d8a02afa196c2b6739255bb0f36c1ff602f853a6 it should be fixed

MuhammadSawalhy commented 2 years ago

after https://github.com/xeluxee/competitest.nvim/commit/d8a02afa196c2b6739255bb0f36c1ff602f853a6 it should be fixed

Well done! Fixed ❤️