playframework#1457 fix(i-am-a-developer): fix test_jvm_version_flag

xael-fry · Feb 11, 2024 · 19fa7db · 19fa7db
1 parent eb3670c
commit 19fa7db
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/framework/build.xml b/framework/build.xml
@@ -268,11 +268,11 @@
 
  <!--exec executable="${pythonExecutable}" failonerror="true">
  <arg value="${basedir}/../samples-and-tests/i-am-a-developer/tests.py" />
- </exec>
+ </exec-->
 
  <exec executable="${pythonExecutable}" failonerror="true">
  <arg value="${basedir}/../samples-and-tests/i-am-a-developer/test_jvm_version_flag.py" />
- </exec-->
+ </exec>
 
  <echo message="Using ${basedir}/../play${playExtension}" />
 

diff --git a/samples-and-tests/i-am-a-developer/test_jvm_version_flag.py b/samples-and-tests/i-am-a-developer/test_jvm_version_flag.py
@@ -33,7 +33,7 @@ def testWithFlag(self, mock):
  play_app.java_cmd([])
 
  step('Assert getJavaVersion was not called')
- self.assert_(not mock.called)
+ mock.assert_(not mock.called)
 
  @mock.patch('play.application.getJavaVersion', return_value='')
  def testWithoutFlag(self, mock):
@@ -43,7 +43,7 @@ def testWithoutFlag(self, mock):
  play_app.java_cmd([])
 
  step('Assert getJavaVersion was called once')
- self.assert_(mock.called)
+ mock.assert_(mock.called)
 
 
 if __name__ == '__main__':