🚑 Fixed bug that prevented users from install Pip requirements

chronos-ui/src/components/ScriptEditor.vue

@@ -42,7 +42,7 @@
       <h3><strong>Pip requirements</strong></h3>
 
       <b-field label="requirements.txt">
-          <b-input type="textarea" v-modal="local_script.requirements"></b-input>
+          <b-input type="textarea" v-model="local_script.requirements"></b-input>
       </b-field>
     </div>
 
@@ -174,7 +174,8 @@ export default {
         name: this.local_script.name,
         interval: this.local_script.interval,
         enabled: this.local_script.enabled,
-        contents: this.local_script.contents
+        contents: this.local_script.contents,
+        requirements: this.local_script.requirements
       }).then(response => {
 
       })

chronos/script.py

@@ -90,9 +90,19 @@ def to_dict(self):
 
     def install_requirements(self):
         """Install requirements.txt"""
-        process = subprocess.check_output(['bash', self.install_requirements_path], universal_newlines=True)
+        process = Popen(['bash', self.install_requirements_path], stdin=PIPE, stdout=PIPE, stderr=PIPE,
+        bufsize=-1)
+
+        output, error = process.communicate()
+
+        # Check if it errored or was successful
+        process_output = ''
+        if process.returncode == 0:
+            process_output = output
+        else:
+            process_output = error
 
-        return process
+        return process_output.decode('utf-8')
 
     def execute(self):
         """Execute script"""

tests/test_database.py

@@ -0,0 +1,2 @@
+def test_answer():
+    assert 5 == 5