> On Oct. 23, 2017, 7:47 p.m., David McLaughlin wrote: > > ui/src/main/js/pages/Instance.js > > Line 34 (original), 38 (patched) > > <https://reviews.apache.org/r/63221/diff/1/?file=1865800#file1865800line39> > > > > enzyme allows you to update props, you can most likely test that when > > you update the URL that the mock APIs are called (see apiSpy in the > > existing tests).
The test that I added checks of the api's `getTasksWithoutConfigs` is called once new params are provided. > On Oct. 23, 2017, 7:47 p.m., David McLaughlin wrote: > > ui/src/main/js/pages/Instance.js > > Lines 55 (patched) > > <https://reviews.apache.org/r/63221/diff/1/?file=1865800#file1865800line56> > > > > I don't think this is related to the previous check. They should be > > independent if statements. Make sense! done! > On Oct. 23, 2017, 7:47 p.m., David McLaughlin wrote: > > ui/src/main/js/pages/Instance.js > > Lines 56 (patched) > > <https://reviews.apache.org/r/63221/diff/1/?file=1865800#file1865800line57> > > > > Note that params is an object. So this is just testing for reference > > equality - this means you could have both false negatives and false > > positives here. You probably want something like the npm module > > "deep-equal". Done. > On Oct. 23, 2017, 7:47 p.m., David McLaughlin wrote: > > ui/src/main/js/pages/Instance.js > > Lines 58 (patched) > > <https://reviews.apache.org/r/63221/diff/1/?file=1865800#file1865800line59> > > > > Extra initialState to a variable and use it here and in constructor? Done! - Reza ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63221/#review188979 ----------------------------------------------------------- On Oct. 23, 2017, 6:01 p.m., Reza Motamedi wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/63221/ > ----------------------------------------------------------- > > (Updated Oct. 23, 2017, 6:01 p.m.) > > > Review request for Aurora, David McLaughlin and Kai Huang. > > > Repository: aurora > > > Description > ------- > > reload instance page once a neighbor item is clicked > > > Diffs > ----- > > ui/src/main/js/pages/Instance.js 4b855457c36e180c0ade629f58e8a03b1273a67a > > > Diff: https://reviews.apache.org/r/63221/diff/1/ > > > Testing > ------- > > Checked in my vagrant and now it works. > > ``` > ? ./gradlew ui:lint > ? ./gradlew ui:test > ``` > > > Thanks, > > Reza Motamedi > >