Skip to content

Better error handling for key extraction #156

Better error handling for key extraction

Better error handling for key extraction #156

GitHub Actions / Unit Test Results failed Sep 27, 2023 in 0s

1 fail, 9 pass in 1m 11s

10 tests   9 ✔️  1m 11s ⏱️
  1 suites  0 💤
  1 files    1

Results for commit 9337e0f.

Annotations

Check warning on line 0 in tests.ExtractFilterArchive.test_ExtractFilterArchive

See this annotation in the file changed.

@github-actions github-actions / Unit Test Results

test_ExtractFilterArchive (tests.ExtractFilterArchive.test_ExtractFilterArchive) failed

junit/test-results.xml
Raw output
def test_ExtractFilterArchive():
        # gets current path
        path = pathlib.Path(__file__).parents[0].absolute()
        sys.path.insert(1, str(path.parents[0]))
    
        # gets the number of cores on the machine
        cores = mp.cpu_count()
        if cores == 1:
            warnings.warn("There is only 1 core on the machine", stacklevel=3)
        else:
            # Remove any old test files/dirs
            if (path / "BP_Extract").exists():
                shutil.rmtree(path / "BP_Extract")
            if (path / ".BP_Extract").exists():
                os.remove(path / ".BP_Extract")
            if (path / ".BP_Extract_BPL").exists():
                os.remove(path / ".BP_Extract_BPL")
            if (path / "BP_Extract.bpa").exists():
                os.remove(path / "BP_Extract.bpa")
            if (path / "Test.bpf").exists():
                os.remove(path / "Test.bpf")
            if (path / "BP_Extract.md5").exists():
                os.remove(path / "BP_Extract.md5")
    
            # Run vspace
            print("Running vspace.")
            sys.stdout.flush()
            subprocess.check_output(["vspace", "vspace.in"], cwd=path)
    
            # Run multi-planet
            print("Running MultiPlanet.")
            sys.stdout.flush()
            subprocess.check_output(["multiplanet", "vspace.in"], cwd=path)
    
            # Run bigplanet
            print("Creating BigPlanet archive.")
            sys.stdout.flush()
            subprocess.check_output(["bigplanet", "bpl.in", "-a"], cwd=path)
    
            # Run bigplanet
            print("Creating BigPlanet file.")
            sys.stdout.flush()
>           subprocess.check_output(["bigplanet", "bpl.in"], cwd=path)

tests/ExtractFilterArchive/test_ExtractFilterArchive.py:54: 
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ 
/usr/share/miniconda/envs/vplanet/lib/python3.7/subprocess.py:411: in check_output
    **kwargs).stdout
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ 

input = None, capture_output = False, timeout = None, check = True
popenargs = (['bigplanet', 'bpl.in'],)
kwargs = {'cwd': PosixPath('/home/runner/work/bigplanet/bigplanet/tests/ExtractFilterArchive'), 'stdout': -1}
process = <subprocess.Popen object at 0x7ffbefe97710>
stdout = b"Creating BPF file...\nFolder Name: BP_Extract\nBPL Archive File: BP_Extract.bpa\nInclude List: ['earth:Instellation:... ['earth.in', 'sun.in']\nPrimary File: vpl.in\nExtracting data from BPA File. Please wait...\nMD5 Checksum verified.\n"
stderr = None, retcode = 1

    def run(*popenargs,
            input=None, capture_output=False, timeout=None, check=False, **kwargs):
        """Run command with arguments and return a CompletedProcess instance.
    
        The returned instance will have attributes args, returncode, stdout and
        stderr. By default, stdout and stderr are not captured, and those attributes
        will be None. Pass stdout=PIPE and/or stderr=PIPE in order to capture them.
    
        If check is True and the exit code was non-zero, it raises a
        CalledProcessError. The CalledProcessError object will have the return code
        in the returncode attribute, and output & stderr attributes if those streams
        were captured.
    
        If timeout is given, and the process takes too long, a TimeoutExpired
        exception will be raised.
    
        There is an optional argument "input", allowing you to
        pass bytes or a string to the subprocess's stdin.  If you use this argument
        you may not also use the Popen constructor's "stdin" argument, as
        it will be used internally.
    
        By default, all communication is in bytes, and therefore any "input" should
        be bytes, and the stdout and stderr will be bytes. If in text mode, any
        "input" should be a string, and stdout and stderr will be strings decoded
        according to locale encoding, or by "encoding" if set. Text mode is
        triggered by setting any of text, encoding, errors or universal_newlines.
    
        The other arguments are the same as for the Popen constructor.
        """
        if input is not None:
            if kwargs.get('stdin') is not None:
                raise ValueError('stdin and input arguments may not both be used.')
            kwargs['stdin'] = PIPE
    
        if capture_output:
            if kwargs.get('stdout') is not None or kwargs.get('stderr') is not None:
                raise ValueError('stdout and stderr arguments may not be used '
                                 'with capture_output.')
            kwargs['stdout'] = PIPE
            kwargs['stderr'] = PIPE
    
        with Popen(*popenargs, **kwargs) as process:
            try:
                stdout, stderr = process.communicate(input, timeout=timeout)
            except TimeoutExpired as exc:
                process.kill()
                if _mswindows:
                    # Windows accumulates the output in a single blocking
                    # read() call run on child threads, with the timeout
                    # being done in a join() on those threads.  communicate()
                    # _after_ kill() is required to collect that and add it
                    # to the exception.
                    exc.stdout, exc.stderr = process.communicate()
                else:
                    # POSIX _communicate already populated the output so
                    # far into the TimeoutExpired exception.
                    process.wait()
                raise
            except:  # Including KeyboardInterrupt, communicate handled that.
                process.kill()
                # We don't call process.wait() as .__exit__ does that for us.
                raise
            retcode = process.poll()
            if check and retcode:
                raise CalledProcessError(retcode, process.args,
>                                        output=stdout, stderr=stderr)
E               subprocess.CalledProcessError: Command '['bigplanet', 'bpl.in']' returned non-zero exit status 1.

/usr/share/miniconda/envs/vplanet/lib/python3.7/subprocess.py:512: CalledProcessError