Re: JS Test error on Windows - Mailing list pgadmin-hackers

From Aditya Toshniwal
Subject Re: JS Test error on Windows
Date
Msg-id CAM9w-_=Apw6tLwjwf2EmuuXGfwkp5FMp12S-6FJVgO6bzhwvAQ@mail.gmail.com
Whole thread Raw
In response to Re: JS Test error on Windows  (Dave Page <dpage@pgadmin.org>)
Responses Re: JS Test error on Windows  (Dave Page <dpage@pgadmin.org>)
List pgadmin-hackers
Hi Dave,

The above error was not thrown when import 'jquery.event.drag' was added to column_selector_spec.js (although it was giving some other error, but above was cleared).
So the doubt was on webpack as it was not importing jquery.event.drag even though it was added using imports-loader, the below test:
        test: /.*slickgrid\/+slick\.(?!core)*/,
        loader: 'imports-loader?' +
        'jquery.ui' +
        ',jquery.event.drag' +
        ',slickgrid',

I noticed later that the test regex is not just for files but also included path. So, gave try by replacing "\/" with "\\" and it worked.
But then this will not work on Unix, and so changed it to [\\\/]+ taking both styles.



Thanks and Regards,
Aditya Toshniwal
Software Engineer | EnterpriseDB Software Solutions | Pune
"Don't Complain about Heat, Plant a tree"

On Fri, May 11, 2018 at 3:05 PM, Dave Page <dpage@pgadmin.org> wrote:


On Fri, May 11, 2018 at 7:28 AM, Aditya Toshniwal <aditya.toshniwal@enterprisedb.com> wrote:
Hi Dave/Hackers,

PFA fix for the problem. The RegEx for webpack test is not working on windows because the path separator is different for Windows( \ ) and Unix( / ). And so, the imports loader is not executing for slickgrid.
Patch tested on Win 10, Win 7, Ubuntu, Cent OS and works fine.

Nice catch! How did you figure that one out?

Patch applied.
 
--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

pgadmin-hackers by date:

Previous
From: Dave Page
Date:
Subject: Re: JS Test error on Windows
Next
From: Dave Page
Date:
Subject: Re: JS Test error on Windows