PermissionBasedStatusMergerSpec is failing

2016-09-08 Thread Tijo Thomas
Hi Nifi test case is failing (PermissionBasedStatusMergerSpec) . This is written in Grovy .. not comfortable with Groovy . Running org.apache.nifi.cluster.manager.PermissionBasedStatusMergerSpec Tests run: 20, Failures: 2, Errors: 0, Skipped: 0, Time elapsed: 0.922 sec <<< FAILURE! - in org.apache

Re: PermissionBasedStatusMergerSpec is failing

2016-09-09 Thread Jeff
Tijo, I just ran this test on master and it's passing for me. Can you provide some details about the branch you're on when running the tests? I see that tasksDuration is 00:30:00.000 when it's expecting 00:00:00.000, and that's why the JSON isn't matching. On Thu, Sep 8, 2016 at 4:58 PM Tijo Th

Re: PermissionBasedStatusMergerSpec is failing

2016-09-10 Thread Tijo Thomas
Hi Jeff I recently rebase from master. Then I cloned again and ran mvn package Tijo On 09-Sep-2016 9:12 pm, "Jeff" wrote: > Tijo, > > I just ran this test on master and it's passing for me. Can you provide > some details about the branch you're on when running the tests? I see that > task

Re: PermissionBasedStatusMergerSpec is failing

2016-09-10 Thread Jeff
Tijo, Have you modified ProcessorStatusSnapshotDTO.java or PermissionBasedStatusMergerSpec.groovy? On Sat, Sep 10, 2016 at 7:48 AM Tijo Thomas wrote: > Hi Jeff > > I recently rebase from master. > Then I cloned again and ran mvn package > > Tijo > > On 09-Sep-2016 9:12 pm, "Jeff" wrote: > >

Re: PermissionBasedStatusMergerSpec is failing

2016-09-13 Thread Tijo Thomas
Sorry to reply late. I was on vacation for last 4 days. I have not modified any files. I think there is some problem with my repo. I will make a new repo and try again. Still the problem exist I will post it again in the group. Thank you very much for your support. Tijo On 10-Sep-2016 6:44

Re: PermissionBasedStatusMergerSpec is failing

2016-09-14 Thread Jeff
Ok, sounds good! Please let us know! On Wed, Sep 14, 2016 at 12:04 AM Tijo Thomas wrote: > Sorry to reply late. I was on vacation for last 4 days. > > I have not modified any files. > > I think there is some problem with my repo. I will make a new repo and > try again. Still the problem exi

Re: PermissionBasedStatusMergerSpec is failing

2016-09-14 Thread Tijo Thomas
Hi Jeff, I tried replacing the mvn repo and did mvn clean package. I am still getting this error. I am sorry I am not getting any clue why it is failing. Can you check again. Tijo On Wed, Sep 14, 2016 at 9:27 PM, Jeff wrote: > Ok, sounds good! Please let us know! > > On Wed, Sep 14, 2016

Re: PermissionBasedStatusMergerSpec is failing

2016-09-15 Thread Jeff
Hello, Could you try building a fresh clone of the NIFI repo? What is your dev environment? What versions of java and maven are you running? Does PermissionBasedStatusMergerSpec work when you run it directly via maven and/or an IDE? On Wed, Sep 14, 2016 at 2:37 PM Tijo Thomas wrote: > Hi Jef

Re: PermissionBasedStatusMergerSpec is failing

2016-09-16 Thread Tijo Thomas
Output when I ran through IDE, Condition not satisfied: returnedJson == expectedJson || | || {"id":"hidden","groupId":"hidden","name":"hidden","type":"hidden","bytesRead":0,"bytesWritten":0,"read":"0 bytes","written":"0 bytes","flowFilesIn":0,"bytesIn":0,"input":"0 (0 by

Re: PermissionBasedStatusMergerSpec is failing

2016-09-16 Thread Jeff
Thank you for the information. Did you try running the tests on a fresh clone of the github repo? Could you please link me to or include the contents of StatusMerger.java and ProcessorStatusSnapshotDTO.java? On Fri, Sep 16, 2016 at 4:17 AM Tijo Thomas wrote: > > Output when I ran through IDE,

Re: PermissionBasedStatusMergerSpec is failing

2016-09-16 Thread Tijo Thomas
Hi Jeff, Yes, I took a fresh clone from git hub . I cleaned my maven repo as well before building . Attached StatusMerger.java and ProcessorStatusSnapshotDTO.java All, It will be great if others also check if it is happening when u build. I am worried whether I am doing some thing really stu

Re: PermissionBasedStatusMergerSpec is failing

2016-09-16 Thread Jeff
I looked into the merger code and found a possible issue. I haven't been able to reproduce the test failure you're getting by setting my locale to en_IN, but I have a sneaking suspicion that FormatUtils might be the culprit here when it's formatting the value for TasksDuration in the target DTO.

Re: PermissionBasedStatusMergerSpec is failing

2016-09-17 Thread Tijo Thomas
Great..   I will  try this tomorrow and let u know.   Tijo Sent from Yahoo Mail on Android On Fri, 16 Sep, 2016 at 11:31 pm, Jeff wrote: I looked into the merger code and found a possible issue.  I haven't been able to reproduce the test failure you're getting by setting my locale to en_I

Re: PermissionBasedStatusMergerSpec is failing

2016-09-18 Thread Tijo Thomas
Hi , I tried replacing with  Locale.getDefault(). Still getting the same error. Sorry Tijo On Saturday, 17 September 2016 8:43 PM, Tijo Thomas wrote: Great..   I will  try this tomorrow and let u know.   Tijo Sent from Yahoo Mail on Android On Fri, 16 Sep, 2016 at 11:31 pm,