From ac91c1eb1724f403bcfae71006b248f8474ad333 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Linus=20=C3=96stberg?= <linus.ostberg@scilifelab.uu.se>
Date: Thu, 15 Aug 2019 10:28:04 +0200
Subject: [PATCH] Add row/block specific disables of pylint tests.

---
 backend/modules/browser/browser_handlers.py |  3 ++-
 backend/modules/browser/lookups.py          | 18 ++++++++----------
 2 files changed, 10 insertions(+), 11 deletions(-)

diff --git a/backend/modules/browser/browser_handlers.py b/backend/modules/browser/browser_handlers.py
index bf7d9c412..568064cd9 100644
--- a/backend/modules/browser/browser_handlers.py
+++ b/backend/modules/browser/browser_handlers.py
@@ -34,7 +34,7 @@ def get(self, dataset: str, query: str, ds_version: str = None):
 class Download(handlers.UnsafeHandler):
     """Download variants in CSV format."""
 
-    def get(self, dataset: str, datatype: str, item: str,
+    def get(self, dataset: str, datatype: str, item: str,  # pylint: disable=too-many-arguments
             ds_version: str = None, filter_type: str = None):
         """
         Download variants in CSV format.
@@ -283,6 +283,7 @@ def get(self, dataset: str, variant: str, ds_version: str = None):
             dataset (str): short name of the dataset
             variant (str): variant in the format chrom-pos-ref-alt
         """
+        # pylint: disable=too-many-locals,too-many-branches,too-many-statements
         dataset, ds_version = utils.parse_dataset(dataset, ds_version)
 
         ret = {'variant': {}}
diff --git a/backend/modules/browser/lookups.py b/backend/modules/browser/lookups.py
index ac7344c8c..d538fab57 100644
--- a/backend/modules/browser/lookups.py
+++ b/backend/modules/browser/lookups.py
@@ -69,6 +69,7 @@ def get_awesomebar_result(dataset: str, query: str, ds_version: str = None) -> t
         tuple: (datatype, identifier)
 
     """
+    # pylint: disable=too-many-return-statements,too-many-branches
     query = query.strip()
 
     # Parse Variant types
@@ -84,35 +85,32 @@ def get_awesomebar_result(dataset: str, query: str, ds_version: str = None) -> t
     # Gene
     try:
         gene = get_gene_by_name(dataset, query)
+        return 'gene', gene['gene_id']
     except error.NotFoundError:
         pass
-    else:
-        return 'gene', gene['gene_id']
+
     # Capital letters for all other queries
     query = query.upper()
     try:
         gene = get_gene_by_name(dataset, query)
+        return 'gene', gene['gene_id']
     except error.NotFoundError:
         pass
-    else:
-        return 'gene', gene['gene_id']
 
     # Ensembl formatted queries
     if query.startswith('ENS'):
         # Gene
         try:
             gene = get_gene(dataset, query)
+            return 'gene', gene['gene_id']
         except error.NotFoundError:
             pass
-        else:
-            return 'gene', gene['gene_id']
         # Transcript
         try:
             transcript = get_transcript(dataset, query)
+            return 'transcript', transcript['transcript_id']
         except error.NotFoundError:
             pass
-        else:
-            return 'transcript', transcript['transcript_id']
 
     # Region and variant queries
     query = query[3:] if query.startswith('CHR') else query
@@ -348,7 +346,7 @@ def get_genes_in_region(dataset: str, chrom: str, start_pos: int,
     return genes
 
 
-def get_raw_variant(dataset: str, pos: int, chrom: str, ref: str,
+def get_raw_variant(dataset: str, pos: int, chrom: str, ref: str,  # pylint: disable=too-many-arguments
                     alt: str, ds_version: str = None) -> dict:
     """
     Retrieve variant by position and change.
@@ -480,7 +478,7 @@ def get_transcripts_in_gene_by_dbid(gene_dbid: int) -> list:
                                           .dicts())]
 
 
-def get_variant(dataset: str, pos: int, chrom: str, ref: str,
+def get_variant(dataset: str, pos: int, chrom: str, ref: str,  # pylint: disable=too-many-arguments
                 alt: str, ds_version: str = None) -> dict:
     """
     Retrieve variant by position and change.