From dfa62d7db93abb4b57fd2e923115b47c0a18f381 Mon Sep 17 00:00:00 2001
From: Harsha Kethineni <harshakethineni@flywheel.io>
Date: Wed, 20 Sep 2017 15:25:52 -0500
Subject: [PATCH] test project filter
---
api/handlers/reporthandler.py | 2 +-
test/integration_tests/python/test_reports.py | 21 +++++++++++++++++++
2 files changed, 22 insertions(+), 1 deletion(-)
diff --git a/api/handlers/reporthandler.py b/api/handlers/reporthandler.py
index fe6fc2ca..7ed79b83 100644
--- a/api/handlers/reporthandler.py
+++ b/api/handlers/reporthandler.py
@@ -505,7 +505,7 @@ class AccessLogReport(Report):
uid = params.get('user')
limit= params.get('limit', 100)
subject = params.get('subject', None)
- project = parmas.get('project', None)
+ project = params.get('project', None)
access_types = params.getall('access_type')
if start_date:
diff --git a/test/integration_tests/python/test_reports.py b/test/integration_tests/python/test_reports.py
index 4eaf7540..548e7369 100644
--- a/test/integration_tests/python/test_reports.py
+++ b/test/integration_tests/python/test_reports.py
@@ -185,6 +185,27 @@ def test_access_log_report(data_builder, with_user, as_user, as_admin):
r = as_admin.get('/report/accesslog', params={'csv': 'true'})
assert r.ok
+ # Create new project with same subject name
+ project2 = data_builder.create_project()
+ r = as_admin.post('/sessions', json={
+ 'project': project2,
+ 'label': 'test-accesslog-session-2',
+ 'timestamp': '1979-01-01T00:00:00+00:00',
+ 'subject': {'code': 'compliant5'}
+ })
+ assert r.ok
+ session2 = r.json()['_id']
+
+ r = as_admin.delete('/sessions/' + session2)
+ data_builder.delete_project(project2, recursive=True)
+
+ # get access log report project filter
+ r = as_admin.get('/report/accesslog', params={'project':project2, 'subject': 'compliant5'})
+ assert r.ok
+
+ for log in r.json():
+ assert log.get('context',{}).get('session',{}).get('label') != 'test-accesslog-session'
+
r.content[0][:3] == '_id'
# get the access types
--
GitLab