search

nus-oss-test / testrepo4

TEAMMATES system is online at
http://teammatesv4.appspot.com
0 stars 0 forks source link

instructorStudentRecordsPageUiTest fails #1765

Closed damithc closed 10 years ago

damithc commented 10 years ago

From franklin...@gmail.com on March 19, 2014 19:36:52

What steps will reproduce the problem? 1.run this test or all the tests. expected output does not match actual html (Commit: Issue 814 ...)

Original issue: http://code.google.com/p/teammatespes/issues/detail?id=1732

damithc commented 10 years ago

From franklin...@gmail.com on March 19, 2014 09:27:31

at org.junit.Assert.assertEquals(Assert.java:115) at org.junit.Assert.assertEquals(Assert.java:144) at teammates.test.driver.HtmlHelper.assertSameHtml(HtmlHelper.java:31) at teammates.test.pageobjects.AppPage.verifyHtml(AppPage.java:485) at teammates.test.cases.ui.browsertests.InstructorStudentRecordsPageUiTest.testContent(InstructorStudentRecordsPageUiTest.java:63) at teammates.test.cases.ui.browsertests.InstructorStudentRecordsPageUiTest.testAll(InstructorStudentRecordsPageUiTest.java:39) at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57) at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) at java.lang.reflect.Method.invoke(Method.java:606) at org.testng.internal.MethodInvocationHelper.invokeMethod(MethodInvocationHelper.java:84) at org.testng.internal.Invoker.invokeMethod(Invoker.java:714) at org.testng.internal.Invoker.invokeTestMethod(Invoker.java:901) at org.testng.internal.Invoker.invokeTestMethods(Invoker.java:1231) at org.testng.internal.TestMethodWorker.invokeTestMethods(TestMethodWorker.java:127) at org.testng.internal.TestMethodWorker.run(TestMethodWorker.java:111) at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145) at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615) at java.lang.Thread.run(Thread.java:744)

testng trace. But Kai did pass the tests on his machine

damithc commented 10 years ago

From dam...@gmail.com on March 19, 2014 16:50:44

We need to see what exactly is different between the actual and expected. May be post a screenshot?

damithc commented 10 years ago

From franklin...@gmail.com on March 19, 2014 22:17:17

Attachment: Untitled.png

damithc commented 10 years ago

From franklin...@gmail.com on March 19, 2014 22:18:15

Attachment: Untitled.png

damithc commented 10 years ago

From dam...@gmail.com on March 19, 2014 22:51:11

It is passing in my comp :-p

damithc commented 10 years ago

From franklin...@gmail.com on March 19, 2014 23:07:22

I will check more on that. But I can pass this test even if I run the default branch. Kai also verified on that he could pass(of course, he committed the latest commit;)). I will go back and check on that. But I guess it is more like the issue with input file or something. Just to confirm, your comp is windows right?

damithc commented 10 years ago

From dam...@gmail.com on March 19, 2014 23:27:55

Yes, Win7.

damithc commented 10 years ago

From franklin...@gmail.com on March 22, 2014 07:10:03

Kai helped compare output and find that deleteComment() js function does not prompt a confirm window on my comp. this may be the setting with my comp although i am using FF12.0 or this could also be the reason of js compatibility...

damithc commented 10 years ago

From franklin...@gmail.com on March 22, 2014 07:18:28

i cannot replicate this for now--the confirm works now for some reason and Dr Damith, you can just close this issue now. sorry for this

damithc commented 10 years ago

From dam...@gmail.com on March 22, 2014 07:28:45

No worries. False alarms are better than undetected errors. :-)

Status: Invalid