dxpy 0.385.0__tar.gz → 0.387.0__tar.gz

{dxpy-0.385.0 → dxpy-0.387.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: dxpy
-Version: 0.385.0
+Version: 0.387.0
 Summary: DNAnexus Platform API bindings for Python
 Home-page: https://github.com/dnanexus/dx-toolkit
 Author: Aleksandra Zalcman, Andrey Kislyuk, Anurag Biyani, Geet Duggal, Katherine Lai, Kurt Jensen, Marek Hrvol, Ohad Rodeh, Phil Sung

{dxpy-0.385.0 → dxpy-0.387.0}/dxpy/bindings/dxproject.py

@@ -285,6 +285,7 @@ class DXProject(DXContainer):
             restricted=None, download_restricted=None, contains_phi=None,
             tags=None, properties=None, bill_to=None, database_ui_view_only=None,
             external_upload_restricted=None, default_symlink=None,
+            database_results_restricted=None,
             **kwargs):
         """
         :param name: The name of the project
@@ -313,6 +314,8 @@ class DXProject(DXContainer):
         :type database_ui_view_only: boolean
         :param external_upload_restricted: If provided, whether project members can upload data to project from external sources, e.g. outside of job
         :type external_upload_restricted: boolean
+        :param database_results_restricted: If provided, minimum amount of data that project members with VIEW access can see from databases in the project
+        :type database_results_restricted: int
         :param default_symlink: If provided, the details needed to have writable symlinks in the project. Dict must include drive, container, and optional prefix.
         :type default_symlink: dict
 
@@ -346,6 +349,8 @@ class DXProject(DXContainer):
             input_hash["databaseUIViewOnly"] = database_ui_view_only
         if external_upload_restricted is not None:
             input_hash["externalUploadRestricted"] = external_upload_restricted
+        if database_results_restricted is not None:
+            input_hash["databaseResultsRestricted"] = database_results_restricted
         if tags is not None:
             input_hash["tags"] = tags
         if properties is not None:
@@ -360,7 +365,9 @@ class DXProject(DXContainer):
     def update(self, name=None, summary=None, description=None, protected=None,
                restricted=None, download_restricted=None, version=None,
                allowed_executables=None, unset_allowed_executables=None,
-               database_ui_view_only=None, external_upload_restricted=None, **kwargs):
+               database_ui_view_only=None, external_upload_restricted=None,
+               database_results_restricted=None, unset_database_results_restricted=None,
+               **kwargs):
         """
         :param name: If provided, the new project name
         :type name: string
@@ -376,10 +383,16 @@ class DXProject(DXContainer):
         :type download_restricted: boolean
         :param allowed_executables: If provided, these are the only executable ID(s) allowed to run as root executions in this project
         :type allowed_executables: list
+        :param unset_allowed_executables: If provided, removes any restrictions set by allowed_executables
+        :type unset_allowed_executables: boolean
         :param database_ui_view_only: If provided, whether the viewers on the project can access the database data directly
         :type database_ui_view_only: boolean
         :param external_upload_restricted: If provided, whether project members can upload data to project from external sources, e.g. outside of job
         :type external_upload_restricted: boolean
+        :param database_results_restricted: If provided, minimum amount of data that project members with VIEW access can see from databases in the project
+        :type database_results_restricted: int
+        :param unset_database_results_restricted: If provided, removes any restrictions set by database_results_restricted
+        :type unset_database_results_restricted: boolean
         :param version: If provided, the update will only occur if the value matches the current project's version number
         :type version: int
 
@@ -413,6 +426,10 @@ class DXProject(DXContainer):
             update_hash["databaseUIViewOnly"] = database_ui_view_only
         if external_upload_restricted is not None:
             update_hash["externalUploadRestricted"] = external_upload_restricted
+        if database_results_restricted is not None:
+            update_hash["databaseResultsRestricted"] = database_results_restricted
+        if unset_database_results_restricted is not None:
+            update_hash["databaseResultsRestricted"] = None
         dxpy.api.project_update(self._dxid, update_hash, **kwargs)
 
     def invite(self, invitee, level, send_email=True, **kwargs):

{dxpy-0.385.0 → dxpy-0.387.0}/dxpy/cli/exec_io.py

@@ -22,7 +22,7 @@ from __future__ import print_function, unicode_literals, division, absolute_impo
 
 # TODO: refactor all dx run helper functions here
 
-import os, sys, json, collections, pipes
+import os, sys, json, collections, shlex
 from ..bindings.dxworkflow import DXWorkflow
 
 import dxpy
@@ -327,7 +327,7 @@ def format_choices_or_suggestions(header, items, obj_class, initial_indent=' ' *
         # TODO: in interactive prompts the quotes here may be a bit
         # misleading. Perhaps it should be a separate mode to print
         # "interactive-ready" suggestions.
-        return fill(header + ' ' + ', '.join([pipes.quote(str(item)) for item in items]),
+        return fill(header + ' ' + ', '.join([shlex.quote(str(item)) for item in items]),
                     initial_indent=initial_indent,
                     subsequent_indent=subsequent_indent)
 

{dxpy-0.385.0 → dxpy-0.387.0}/dxpy/cli/parsers.py

@@ -446,6 +446,10 @@ def get_update_project_args(args):
         input_params['allowedExecutables'] = args.allowed_executables
     if args.unset_allowed_executables:
         input_params['allowedExecutables'] = None
+    if args.database_results_restricted is not None:
+        input_params['databaseResultsRestricted'] = args.database_results_restricted
+    if args.unset_database_results_restricted:
+        input_params['databaseResultsRestricted'] = None
     if args.external_upload_restricted is not None:
         input_params['externalUploadRestricted'] = args.external_upload_restricted == 'true'
     return input_params

{dxpy-0.385.0 → dxpy-0.387.0}/dxpy/scripts/dx.py

@@ -1432,6 +1432,8 @@ def new_project(args):
         inputs["containsPHI"] = True
     if args.database_ui_view_only:
         inputs["databaseUIViewOnly"] = True
+    if args.database_results_restricted is not None:
+        inputs["databaseResultsRestricted"] = args.database_results_restricted
     if args.monthly_compute_limit is not None:
         inputs["monthlyComputeLimit"] = args.monthly_compute_limit
     if args.monthly_egress_bytes_limit is not None:
@@ -2479,7 +2481,7 @@ def find_data(args):
                                          visibility=args.visibility,
                                          properties=args.properties,
                                          name=args.name,
-                                         name_mode='glob',
+                                         name_mode=args.name_mode,
                                          typename=args.type,
                                          tags=args.tag, link=args.link,
                                          project=args.project,
@@ -5427,6 +5429,9 @@ parser_update_project.add_argument('--bill-to', help="Update the user or org ID
 allowed_executables_group = parser_update_project.add_mutually_exclusive_group()
 allowed_executables_group.add_argument('--allowed-executables', help='Executable ID(s) this project is allowed to run.  This operation overrides any existing list of executables.', type=str, nargs="+")
 allowed_executables_group.add_argument('--unset-allowed-executables', help='Removes any restriction to run executables as set by --allowed-executables', action='store_true')
+database_results_restricted_group = parser_update_project.add_mutually_exclusive_group()
+database_results_restricted_group.add_argument('--database-results-restricted', help='Viewers on the project can access only more than specified size of visual data from databases', type=positive_integer)
+database_results_restricted_group.add_argument('--unset-database-results-restricted', help='Removes any restriction to return data from databases as set by --database-results-restricted', action='store_true')
 
 parser_update_project.set_defaults(func=update_project)
 register_parser(parser_update_project, subparsers_action=subparsers_update, categories="metadata")
@@ -5813,6 +5818,7 @@ parser_new_project.add_argument('--phi', help='Add PHI protection to project', d
                                 action='store_true')
 parser_new_project.add_argument('--database-ui-view-only', help='Viewers on the project cannot access database data directly', default=False,
                                 action='store_true')
+parser_new_project.add_argument('--database-results-restricted', help='Viewers on the project can access only more than specified size of visual data from databases', type=positive_integer)
 parser_new_project.add_argument('--monthly-compute-limit', type=positive_integer, help='Monthly project spending limit for compute')
 parser_new_project.add_argument('--monthly-egress-bytes-limit', type=positive_integer, help='Monthly project spending limit for egress (in Bytes)')
 parser_new_project.add_argument('--monthly-storage-limit', type=positive_number, help='Monthly project spending limit for storage')
@@ -6137,7 +6143,8 @@ parser_find_data.add_argument('--class', dest='classname', choices=['record', 'f
 )
 parser_find_data.add_argument('--state', choices=['open', 'closing', 'closed', 'any'], help='State of the object')
 parser_find_data.add_argument('--visibility', choices=['hidden', 'visible', 'either'], default='visible', help='Whether the object is hidden or not')
-parser_find_data.add_argument('--name', help='Name of the object')
+parser_find_data.add_argument('--name', help='Search criteria for the object name, interpreted according to the --name-mode')
+parser_find_data.add_argument('--name-mode', default='glob', help='Name mode to use for searching', choices=['glob', 'exact', 'regexp'])
 parser_find_data.add_argument('--type', help='Type of the data object')
 parser_find_data.add_argument('--link', help='Object ID that the data object links to')
 parser_find_data.add_argument('--all-projects', '--allprojects', help='Extend search to all projects (excluding public projects)', action='store_true')

dxpy-0.387.0/dxpy/toolkit_version.py

@@ -0,0 +1 @@
+version = '0.387.0'

{dxpy-0.385.0 → dxpy-0.387.0}/dxpy/utils/__init__.py

@@ -292,10 +292,10 @@ class Nonce:
     '''
     def __init__(self):
         try:
-            self.nonce = "%s%f" % (str(binascii.hexlify(os.urandom(32))), time.time())
+            self.nonce = "%s%f" % (binascii.hexlify(os.urandom(32)).decode('utf-8'), time.time())
         except:
             random.seed(time.time())
-            self.nonce = "%s%f" % (str(random.getrandbits(8*26)), time.time())
+            self.nonce = "%s%f" % (random.getrandbits(8*26), time.time())
 
     def __str__(self):
         return self.nonce

{dxpy-0.385.0 → dxpy-0.387.0}/dxpy/utils/describe.py

@@ -415,7 +415,7 @@ def print_project_desc(desc, verbose=False):
         'containsPHI', 'databaseUIViewOnly', 'externalUploadRestricted', 'region', 'storageCost', 'pendingTransfer',
         'atSpendingLimit', 'currentMonthComputeAvailableBudget', 'currentMonthEgressBytesAvailableBudget',
         'currentMonthStorageAvailableBudget', 'currentMonthComputeUsage', 'currentMonthEgressBytesUsage', 
-        'currentMonthExpectedStorageUsage', 'defaultSymlink'
+        'currentMonthExpectedStorageUsage', 'defaultSymlink', 'databaseResultsRestricted',
         # Following are app container-specific
         'destroyAt', 'project', 'type', 'app', 'appName'
     ]
@@ -455,6 +455,8 @@ def print_project_desc(desc, verbose=False):
         print_json_field('External Upload Restricted', desc['externalUploadRestricted'])
     if 'defaultSymlink' in desc and verbose:
         print_json_field('Default Symlink', desc['defaultSymlink'])
+    if 'databaseResultsRestricted' in desc and desc['databaseResultsRestricted']:
+        print_json_field('Database Results Restricted', desc['databaseResultsRestricted'])
 
     # Usage
     print_field("Created", render_timestamp(desc['created']))

{dxpy-0.385.0 → dxpy-0.387.0}/dxpy/utils/exec_utils.py

@@ -23,7 +23,7 @@ from __future__ import print_function, unicode_literals, division, absolute_impo
 import os, sys, json, re, collections, logging, argparse, string, itertools, subprocess, tempfile
 from functools import wraps
 from collections import namedtuple
-import pipes
+import shlex
 
 import dxpy
 from ..compat import USING_PYTHON2, open, Mapping
@@ -435,7 +435,7 @@ class DXExecDependencyInstaller(object):
                 dxpy.download_dxfile(bundle["id"], bundle["name"], project=dxpy.WORKSPACE_ID)
             except dxpy.exceptions.ResourceNotFound:
                 dxpy.download_dxfile(bundle["id"], bundle["name"])
-            self.run("dx-unpack {}".format(pipes.quote(bundle["name"])))
+            self.run("dx-unpack {}".format(shlex.quote(bundle["name"])))
         else:
             self.log('Skipping bundled dependency "{name}" because it does not refer to a file'.format(**bundle))
 

{dxpy-0.385.0 → dxpy-0.387.0}/dxpy/utils/file_load_utils.py

@@ -83,7 +83,7 @@ will download into the execution environment:
 from __future__ import print_function, unicode_literals, division, absolute_import
 
 import json
-import pipes
+import shlex
 import os
 import fnmatch
 import sys
@@ -401,10 +401,6 @@ def analyze_bash_vars(job_input_file, job_homedir):
     return file_key_descs, rest_hash
 
 
-#
-# Note: pipes.quote() to be replaced with shlex.quote() in Python 3
-# (see http://docs.python.org/2/library/pipes.html#pipes.quote)
-#
 def gen_bash_vars(job_input_file, job_homedir=None, check_name_collision=True):
     """
     :param job_input_file: path to a JSON file describing the job inputs
@@ -427,7 +423,7 @@ def gen_bash_vars(job_input_file, job_homedir=None, check_name_collision=True):
             result = json.dumps(dxpy.dxlink(elem))
         else:
             result = json.dumps(elem)
-        return pipes.quote(result)
+        return shlex.quote(result)
 
     def string_of_value(val):
         if isinstance(val, list):

{dxpy-0.385.0 → dxpy-0.387.0}/dxpy/utils/local_exec_utils.py

@@ -16,7 +16,7 @@
 
 from __future__ import print_function, unicode_literals, division, absolute_import
 
-import os, sys, json, subprocess, pipes
+import os, sys, json, subprocess, shlex
 import collections, datetime
 
 import dxpy
@@ -351,9 +351,9 @@ def run_one_entry_point(job_id, function, input_hash, run_spec, depends_on, name
           if [[ $(type -t {function}) == "function" ]];
           then {function};
           else echo "$0: Global scope execution complete. Not invoking entry point function {function} because it was not found" 1>&2;
-          fi'''.format(homedir=pipes.quote(job_homedir),
-                       env_path=pipes.quote(os.path.join(job_env['HOME'], 'environment')),
-                       code_path=pipes.quote(environ['DX_TEST_CODE_PATH']),
+          fi'''.format(homedir=shlex.quote(job_homedir),
+                       env_path=shlex.quote(os.path.join(job_env['HOME'], 'environment')),
+                       code_path=shlex.quote(environ['DX_TEST_CODE_PATH']),
                        function=function)
         invocation_args = ['bash', '-c', '-e'] + (['-x'] if environ.get('DX_TEST_X_FLAG') else []) + [script]
     elif run_spec['interpreter'] == 'python2.7':

{dxpy-0.385.0 → dxpy-0.387.0}/dxpy.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: dxpy
-Version: 0.385.0
+Version: 0.387.0
 Summary: DNAnexus Platform API bindings for Python
 Home-page: https://github.com/dnanexus/dx-toolkit
 Author: Aleksandra Zalcman, Andrey Kislyuk, Anurag Biyani, Geet Duggal, Katherine Lai, Kurt Jensen, Marek Hrvol, Ohad Rodeh, Phil Sung

{dxpy-0.385.0 → dxpy-0.387.0}/test/test_dx_app_wizard.py

@@ -19,7 +19,6 @@
 
 import os, sys, unittest, json, tempfile, subprocess
 import pexpect
-import pipes
 
 from dxpy_testutil import DXTestCase, check_output
 import dxpy_testutil as testutil

{dxpy-0.385.0 → dxpy-0.387.0}/test/test_dx_bash_helpers.py

@@ -23,7 +23,7 @@ import dxpy
 import dxpy_testutil as testutil
 import json
 import os
-import pipes
+import shlex
 import pytest
 import shutil
 import tempfile
@@ -42,7 +42,7 @@ from dxpy.bindings.download_all_inputs import _get_num_parallel_threads
 def run(command, **kwargs):
     try:
         if isinstance(command, list) or isinstance(command, tuple):
-            print("$ %s" % " ".join(pipes.quote(f) for f in command))
+            print("$ %s" % " ".join(shlex.quote(f) for f in command))
             output = check_output(command, **kwargs)
         else:
             print("$ %s" % (command,))
@@ -919,30 +919,30 @@ class TestDXJobutilNewJob(DXTestCase):
             ),
             # instance type: mapping
             ("--instance-type " +
-                pipes.quote(json.dumps({"main": "mem2_hdd2_x2" , "other_function": "mem2_hdd2_x1" })),
+                shlex.quote(json.dumps({"main": "mem2_hdd2_x2" , "other_function": "mem2_hdd2_x1" })),
                 {"systemRequirements": {"main": { "instanceType": "mem2_hdd2_x2" },
                                         "other_function": { "instanceType": "mem2_hdd2_x1" }}}),
             ("--instance-type-by-executable " +
-                pipes.quote(json.dumps({"my_applet": {"main": "mem2_hdd2_x2",
+                shlex.quote(json.dumps({"my_applet": {"main": "mem2_hdd2_x2",
                                         "other_function": "mem3_ssd2_fpga1_x8"}})),
                 {"systemRequirementsByExecutable": {"my_applet": {"main": {"instanceType": "mem2_hdd2_x2"},
                                                     "other_function": {"instanceType": "mem3_ssd2_fpga1_x8"}}}}),
             ("--instance-type-by-executable " +
-             pipes.quote(json.dumps({"my_applet": {"main": "mem1_ssd1_v2_x2",
+             shlex.quote(json.dumps({"my_applet": {"main": "mem1_ssd1_v2_x2",
                                                    "other_function": "mem3_ssd2_fpga1_x8"}})) +
              " --extra-args " +
-                pipes.quote(json.dumps({"systemRequirementsByExecutable": {"my_applet": {"main": {"instanceType": "mem2_hdd2_x2", "clusterSpec": {"initialInstanceCount": 3}},
+                shlex.quote(json.dumps({"systemRequirementsByExecutable": {"my_applet": {"main": {"instanceType": "mem2_hdd2_x2", "clusterSpec": {"initialInstanceCount": 3}},
                                         "other_function": {"fpgaDriver": "edico-1.4.5"}}}})),
                 {"systemRequirementsByExecutable": {"my_applet":{"main": { "instanceType": "mem2_hdd2_x2", "clusterSpec":{"initialInstanceCount": 3}},
                                         "other_function": { "instanceType": "mem3_ssd2_fpga1_x8", "fpgaDriver": "edico-1.4.5"} }}}),
             # nvidia driver
             ("--instance-type-by-executable " +
-             pipes.quote(json.dumps({
+             shlex.quote(json.dumps({
                  "my_applet": {
                      "main": "mem1_ssd1_v2_x2",
                      "other_function": "mem2_ssd1_gpu_x16"}})) +
              " --extra-args " +
-             pipes.quote(json.dumps({
+             shlex.quote(json.dumps({
                  "systemRequirementsByExecutable": {
                      "my_applet": {
                          "main": {"instanceType": "mem2_hdd2_x2"},
@@ -963,14 +963,14 @@ class TestDXJobutilNewJob(DXTestCase):
             self.assertNewJobInputHash(cmd_snippet, arguments_hash)
 
     def test_extra_arguments(self):
-        cmd_snippet = "--extra-args " + pipes.quote(
+        cmd_snippet = "--extra-args " + shlex.quote(
             json.dumps({"details": {"d1": "detail1", "d2": 1234}, "foo": "foo_value"})
         )
         arguments_hash = {"details": {"d1": "detail1", "d2": 1234}, "foo": "foo_value"}
         self.assertNewJobInputHash(cmd_snippet, arguments_hash)
 
         # override previously specified args
-        cmd_snippet = "--name JobName --extra-args " + pipes.quote(
+        cmd_snippet = "--name JobName --extra-args " + shlex.quote(
             json.dumps({"name": "FinalName"})
         )
         arguments_hash = {"name": "FinalName"}

{dxpy-0.385.0 → dxpy-0.387.0}/test/test_dxclient.py

@@ -21,7 +21,7 @@ from __future__ import print_function, unicode_literals, division, absolute_impo
 
 import os, sys, unittest, json, tempfile, subprocess, shutil, re, base64, random, time
 import filecmp
-import pipes
+import shlex
 import stat
 import hashlib
 import collections
@@ -380,14 +380,14 @@ class TestDXClient(DXTestCase):
 
             # Test -f with valid JSON file.
             record_id = run("dx new record Ψ2 --brief").strip()
-            run("dx set_details Ψ2 -f " + pipes.quote(tmp_file.name))
+            run("dx set_details Ψ2 -f " + shlex.quote(tmp_file.name))
             dxrecord = dxpy.DXRecord(record_id)
             details = dxrecord.get_details()
             self.assertEqual({"foo": "bar"}, details, msg="dx set_details -f with valid JSON input file failed.")
 
             # Test --details-file with valid JSON file.
             record_id = run("dx new record Ψ3 --brief").strip()
-            run("dx set_details Ψ3 --details-file " + pipes.quote(tmp_file.name))
+            run("dx set_details Ψ3 --details-file " + shlex.quote(tmp_file.name))
             dxrecord = dxpy.DXRecord(record_id)
             details = dxrecord.get_details()
             self.assertEqual({"foo": "bar"}, details,
@@ -400,16 +400,16 @@ class TestDXClient(DXTestCase):
             # Test above with invalid JSON file.
             record_id = run("dx new record Ψ4 --brief").strip()
             with self.assertSubprocessFailure(stderr_regexp="JSON", exit_code=3):
-                run("dx set_details Ψ4 -f " + pipes.quote(tmp_invalid_file.name))
+                run("dx set_details Ψ4 -f " + shlex.quote(tmp_invalid_file.name))
 
             # Test command with (-f or --details-file) and CL JSON.
             with self.assertSubprocessFailure(stderr_regexp="Error: Cannot provide both -f/--details-file and details",
                                               exit_code=3):
-                run("dx set_details Ψ4 '{ \"foo\":\"bar\" }' -f " + pipes.quote(tmp_file.name))
+                run("dx set_details Ψ4 '{ \"foo\":\"bar\" }' -f " + shlex.quote(tmp_file.name))
 
             # Test piping JSON from STDIN.
             record_id = run("dx new record Ψ5 --brief").strip()
-            run("cat " + pipes.quote(tmp_file.name) + " | dx set_details Ψ5 -f -")
+            run("cat " + shlex.quote(tmp_file.name) + " | dx set_details Ψ5 -f -")
             dxrecord = dxpy.DXRecord(record_id)
             details = dxrecord.get_details()
             self.assertEqual({"foo": "bar"}, details, msg="dx set_details -f - with valid JSON input failed.")
@@ -5504,11 +5504,11 @@ class TestDXClientFind(DXTestCase):
     def test_dx_find_projects(self):
         unique_project_name = 'dx find projects test ' + str(time.time())
         with temporary_project(unique_project_name) as unique_project:
-            self.assertEqual(run("dx find projects --name " + pipes.quote(unique_project_name)),
+            self.assertEqual(run("dx find projects --name " + shlex.quote(unique_project_name)),
                              unique_project.get_id() + ' : ' + unique_project_name + ' (ADMINISTER)\n')
-            self.assertEqual(run("dx find projects --brief --name " + pipes.quote(unique_project_name)),
+            self.assertEqual(run("dx find projects --brief --name " + shlex.quote(unique_project_name)),
                              unique_project.get_id() + '\n')
-            json_output = json.loads(run("dx find projects --json --name " + pipes.quote(unique_project_name)))
+            json_output = json.loads(run("dx find projects --json --name " + shlex.quote(unique_project_name)))
             self.assertEqual(len(json_output), 1)
             self.assertEqual(json_output[0]['id'], unique_project.get_id())
 
@@ -5529,15 +5529,15 @@ class TestDXClientFind(DXTestCase):
         created_project_name = 'dx find projects test ' + str(time.time())
         with temporary_project(created_project_name) as unique_project:
             self.assertEqual(run("dx find projects --created-after=-1d --brief --name " +
-                             pipes.quote(created_project_name)), unique_project.get_id() + '\n')
+                             shlex.quote(created_project_name)), unique_project.get_id() + '\n')
             self.assertEqual(run("dx find projects --created-before=" + str(int(time.time() + 1000) * 1000) +
-                             " --brief --name " + pipes.quote(created_project_name)),
+                             " --brief --name " + shlex.quote(created_project_name)),
                              unique_project.get_id() + '\n')
             self.assertEqual(run("dx find projects --created-after=-1d --created-before=" +
                              str(int(time.time() + 1000) * 1000) + " --brief --name " +
-                             pipes.quote(created_project_name)), unique_project.get_id() + '\n')
+                             shlex.quote(created_project_name)), unique_project.get_id() + '\n')
             self.assertEqual(run("dx find projects --created-after=" + str(int(time.time() + 1000) * 1000) + " --name "
-                             + pipes.quote(created_project_name)), "")
+                             + shlex.quote(created_project_name)), "")
 
     def test_dx_find_projects_by_region(self):
         awseast = "aws:us-east-1"
@@ -5545,7 +5545,7 @@ class TestDXClientFind(DXTestCase):
         created_project_name = 'dx find projects test ' + str(time.time())
         with temporary_project(created_project_name, region=awseast) as unique_project:
             self.assertEqual(run("dx find projects --region {} --brief --name {}".format(
-                                 awseast, pipes.quote(created_project_name))),
+                                 awseast, shlex.quote(created_project_name))),
                              unique_project.get_id() + '\n')
             self.assertIn(unique_project.get_id(),
                           run("dx find projects --region {} --brief".format(awseast)))
@@ -5635,10 +5635,10 @@ class TestDXClientFind(DXTestCase):
     def test_dx_find_projects_phi(self):
         projectName = "tempProject+{t}".format(t=time.time())
         with temporary_project(name=projectName) as project_1:
-            res = run('dx find projects --phi true --brief --name ' + pipes.quote(projectName))
+            res = run('dx find projects --phi true --brief --name ' + shlex.quote(projectName))
             self.assertTrue(len(res) == 0, "Expected no PHI projects to be found")
 
-            res = run('dx find projects --phi false --brief --name ' + pipes.quote(projectName)).strip().split('\n')
+            res = run('dx find projects --phi false --brief --name ' + shlex.quote(projectName)).strip().split('\n')
             self.assertTrue(len(res) == 1, "Expected to find one project")
             self.assertTrue(res[0] == project_1.get_id())
 
@@ -6287,10 +6287,10 @@ class TestDXClientFindInOrg(DXTestCaseBuildApps):
             project1_id = project_1.get_id()
             dxpy.api.project_update(project1_id, {"billTo": self.org_id})
 
-            res = run('dx find org projects org-piratelabs --phi true --brief --name ' + pipes.quote(projectName))
+            res = run('dx find org projects org-piratelabs --phi true --brief --name ' + shlex.quote(projectName))
             self.assertTrue(len(res) == 0, "Expected no PHI projects to be found")
 
-            res = run('dx find org projects org-piratelabs --phi false --brief --name ' + pipes.quote(projectName)).strip().split("\n")
+            res = run('dx find org projects org-piratelabs --phi false --brief --name ' + shlex.quote(projectName)).strip().split("\n")
 
             self.assertTrue(len(res) == 1, "Expected to find one project")
             self.assertEqual(res[0], project1_id)
@@ -7431,7 +7431,7 @@ class TestDXClientUpdateProject(DXTestCase):
 
         #Update items one by one.
         for item in update_items:
-            run(self.cmd.format(pid=self.project, item=item, n=pipes.quote(update_items[item])))
+            run(self.cmd.format(pid=self.project, item=item, n=shlex.quote(update_items[item])))
             describe_input = {}
             describe_input[item] = 'true'
             self.assertEqual(self.project_describe(describe_input)[item],
@@ -7447,14 +7447,14 @@ class TestDXClientUpdateProject(DXTestCase):
                         'protected': 'false'}
 
         update_project_output = check_output(["dx", "update", "project", self.project, "--name",
-                pipes.quote(update_items['name']), "--summary", update_items['summary'], "--description",
+                shlex.quote(update_items['name']), "--summary", update_items['summary'], "--description",
                 update_items['description'], "--protected", update_items['protected']])
         update_project_json = json.loads(update_project_output);
         self.assertTrue("id" in update_project_json)
         self.assertEqual(self.project, update_project_json["id"])
 
         update_project_output = check_output(["dx", "update", "project", self.project, "--name",
-                pipes.quote(update_items['name']), "--summary", update_items['summary'], "--description",
+                shlex.quote(update_items['name']), "--summary", update_items['summary'], "--description",
                 update_items['description'], "--protected", update_items['protected'], "--brief"])
         self.assertEqual(self.project, update_project_output.rstrip("\n"))
 
@@ -7479,7 +7479,7 @@ class TestDXClientUpdateProject(DXTestCase):
         project_name = self.project_describe(describe_input)['name']
         new_name = 'Another Project Name' + str(time.time())
 
-        run(self.cmd.format(pid=project_name, item='name', n=pipes.quote(new_name)))
+        run(self.cmd.format(pid=project_name, item='name', n=shlex.quote(new_name)))
         result = self.project_describe(describe_input)
         self.assertEqual(result['name'], new_name)
 

{dxpy-0.385.0 → dxpy-0.387.0}/test/test_dxpy.py

@@ -139,6 +139,7 @@ class TestDXProject(unittest.TestCase):
             self.assertEqual(desc["databaseUIViewOnly"], False)
             self.assertEqual(desc["externalUploadRestricted"], False)
             self.assertEqual(desc["tags"], [])
+            self.assertEqual(desc["databaseResultsRestricted"], None)
             prop = dxpy.api.project_describe(dxproject.get_id(),
                                              {'fields': {'properties': True}})
             self.assertEqual(prop['properties'], {})
@@ -192,6 +193,7 @@ class TestDXProject(unittest.TestCase):
                          download_restricted=True,
                          external_upload_restricted=False,
                          allowed_executables=["applet-abcdefghijklmnopqrstuzwx"],
+                         database_results_restricted=10,
                          description="new description")
         desc = dxproject.describe()
         self.assertEqual(desc["id"], self.proj_id)
@@ -203,13 +205,15 @@ class TestDXProject(unittest.TestCase):
         self.assertEqual(desc["externalUploadRestricted"], False)
         self.assertEqual(desc["description"], "new description")
         self.assertEqual(desc["allowedExecutables"][0], "applet-abcdefghijklmnopqrstuzwx")
+        self.assertEqual(desc["databaseResultsRestricted"], 10)
         self.assertTrue("created" in desc)
 
-        dxproject.update(restricted=False, download_restricted=False, unset_allowed_executables=True)
+        dxproject.update(restricted=False, download_restricted=False, unset_allowed_executables=True, unset_database_results_restricted=True)
         desc = dxproject.describe()
         self.assertEqual(desc["restricted"], False)
         self.assertEqual(desc["downloadRestricted"], False)
-        self.assertTrue("allowedExecutables" not in desc)
+        self.assertEqual(desc["allowedExecutables"], None)
+        self.assertEqual(desc["databaseResultsRestricted"], None)
 
     def test_new_list_remove_folders(self):
         dxproject = dxpy.DXProject()

dxpy-0.385.0/dxpy/toolkit_version.py

@@ -1 +0,0 @@
-version = '0.385.0'