myfaces icon indicating copy to clipboard operation
myfaces copied to clipboard

MYFACES-4476: New Faces.js integration tests (target 4.0.0 RC3)

Open werpu opened this issue 3 years ago • 4 comments

I have ported my set of 20 integration tests, i have used for developing the jsf.js and faces.ts scripts in an upgraded version which use the newest client side frameworks for testing.

werpu avatar Oct 11 '22 09:10 werpu

Sorry, i discovered some issues, I am closing the pr and will reissue it again later

werpu avatar Oct 11 '22 10:10 werpu

I have fixed now the codebase, all tests pass on the new files, one fails on the old codebase. For now I have enabled the new faces.js codebase for build and will enable the standard include, once the new codebase is merged. I am not going to fix the failing test on the old codebase anymore. (it is very likely a bug still referencing the old namespace)

werpu avatar Oct 11 '22 14:10 werpu

Running build

melloware avatar Oct 11 '22 14:10 melloware

Integration tests are now basically ready for merge, but before the merge we have to retarget the faces.js include in the imports.xhtml file. Atm it is pointing towards at my embedded github implementation which is the downstream for myfaces. I will spend another few rounts on fixing typos and documentation and code improvement, but it is already mergable.

werpu avatar Oct 18 '22 14:10 werpu