From b5a7162683922405d826a777e2d149a9fc2bd86c Mon Sep 17 00:00:00 2001
From: Samuel Colvin <s@muelcolvin.com>
Date: Thu, 11 Jan 2018 12:19:51 +0000
Subject: [PATCH 1/3] adding labels filter

---
 public/index.html                         | 4 ++++
 src/components/contractors/Contractors.js | 3 ++-
 src/components/enquiry/EnquiryButton.js   | 2 +-
 src/index.js                              | 2 ++
 src/utils.js                              | 9 +++++++--
 5 files changed, 16 insertions(+), 4 deletions(-)

diff --git a/public/index.html b/public/index.html
index 148f956e..f532289f 100644
--- a/public/index.html
+++ b/public/index.html
@@ -37,6 +37,10 @@
       router_mode: 'history',
       api_root: api_root,
       mode: 'grid',
+      // contractor_filter: {
+      //   labels: ['foo', 'bar'],
+      //   label_exclude: ['whatever']
+      // },
       event_callback: function (name, v) {
         console.log(name, v)
       }
diff --git a/src/components/contractors/Contractors.js b/src/components/contractors/Contractors.js
index c4bd5500..628e8090 100644
--- a/src/components/contractors/Contractors.js
+++ b/src/components/contractors/Contractors.js
@@ -49,7 +49,8 @@ class Contractors extends Component {
 
     this.setState({selected_subject})
     const sub_id = selected_subject && selected_subject.id
-    const contractors = await this.props.root.requests.get('contractors', {subject: sub_id || null})
+    const args = Object.assign({}, this.props.config.contractor_filter, {subject: sub_id || null})
+    const contractors = await this.props.root.requests.get('contractors', args)
     this.props.config.event_callback('updated_contractors', contractors)
     this.setState({
       contractors,
diff --git a/src/components/enquiry/EnquiryButton.js b/src/components/enquiry/EnquiryButton.js
index 11110519..cd24e756 100644
--- a/src/components/enquiry/EnquiryButton.js
+++ b/src/components/enquiry/EnquiryButton.js
@@ -4,7 +4,7 @@ import EnquiryModal from './EnquiryModal'
 
 const EnquiryButton = ({root, config}) => (
   <div className="tcs-app">
-    <div class="tcs-enquiry-modal">
+    <div className="tcs-enquiry-modal">
       <Link to={root.url('enquiry')} className="tcs-enquiry-button">
         {root.get_text('enquiry_button')}
       </Link>
diff --git a/src/index.js b/src/index.js
index 1e205c93..81bc486a 100644
--- a/src/index.js
+++ b/src/index.js
@@ -97,6 +97,8 @@ window.socket = function (public_key, config) {
     config.element = '#socket'
   }
 
+  config.contractor_filter = config.contractor_filter || {}
+
   if (config.subject_filter === undefined) {
     config.subject_filter = true
   }
diff --git a/src/utils.js b/src/utils.js
index 58533d40..953c74c5 100644
--- a/src/utils.js
+++ b/src/utils.js
@@ -128,9 +128,14 @@ export const requests = {
   get: async (app, path, args) => {
     if (args) {
       const arg_list = []
+      const add_arg = (n, v) => arg_list.push(encodeURIComponent(n) + '=' + encodeURIComponent(v))
       for (let [name, value] of Object.entries(args)) {
-        if (value !== null) {
-          arg_list.push(encodeURIComponent(name) + '=' + encodeURIComponent(value))
+        if (Array.isArray(value)) {
+          for (let value_ of value) {
+            add_arg(name, value_)
+          }
+        } else if (value !== null && value !== undefined) {
+          add_arg(name, value)
         }
       }
       if (arg_list.length > 0) {

From 2a604bbca5333c2d04664d65f859c640ac260035 Mon Sep 17 00:00:00 2001
From: Samuel Colvin <s@muelcolvin.com>
Date: Thu, 11 Jan 2018 13:03:30 +0000
Subject: [PATCH 2/3] tweaks

---
 public/index.html | 10 ++++------
 src/index.js      | 10 +++++++++-
 2 files changed, 13 insertions(+), 7 deletions(-)

diff --git a/public/index.html b/public/index.html
index f532289f..25907bcf 100644
--- a/public/index.html
+++ b/public/index.html
@@ -30,17 +30,15 @@
     </main>
   </body>
   <script>
-    var public_key = '9c79f14df986a1ec693c'
-    var api_root = null  // 'http://localhost:8000'
+    var public_key = '9c79f14df986a1ec693c'  // '61e9d8d03109e44d7c67'
+    var api_root = null  // 'https://socket-beta.tutorcruncher.com' 'http://localhost:8000'
     var socket1 = socket(public_key, {
       element: '#socket1',
       router_mode: 'history',
       api_root: api_root,
       mode: 'grid',
-      // contractor_filter: {
-      //   labels: ['foo', 'bar'],
-      //   label_exclude: ['whatever']
-      // },
+      labels_include: [],
+      labels_exclude: [],
       event_callback: function (name, v) {
         console.log(name, v)
       }
diff --git a/src/index.js b/src/index.js
index 81bc486a..3c18e926 100644
--- a/src/index.js
+++ b/src/index.js
@@ -97,7 +97,15 @@ window.socket = function (public_key, config) {
     config.element = '#socket'
   }
 
-  config.contractor_filter = config.contractor_filter || {}
+  config.contractor_filter = {}
+  if (config.labels_include) {
+    config.contractor_filter.label = config.labels_include
+    delete config.labels_include
+  }
+  if (config.labels_exclude) {
+    config.contractor_filter.label_exclude = config.labels_exclude
+    delete config.labels_exclude
+  }
 
   if (config.subject_filter === undefined) {
     config.subject_filter = true

From 250cfe8abd05392751bbfa49dbad1b49aed066b5 Mon Sep 17 00:00:00 2001
From: Samuel Colvin <s@muelcolvin.com>
Date: Thu, 11 Jan 2018 14:06:37 +0000
Subject: [PATCH 3/3] tests for contractor filters

---
 .eslintrc.js            |  1 +
 src/index.js            |  3 ++-
 src/tests/App.test.js   | 37 ++++++++++++++++++++++++++++++++++---
 src/tests/index.test.js | 27 ++++++++++++++++++++++++++-
 src/tests/setup.js      |  4 +---
 src/tests/utils.js      | 29 +++++++++++++++++++++--------
 6 files changed, 85 insertions(+), 16 deletions(-)

diff --git a/.eslintrc.js b/.eslintrc.js
index 923f8cfa..16358af0 100644
--- a/.eslintrc.js
+++ b/.eslintrc.js
@@ -9,6 +9,7 @@ module.exports = {
   },
   globals: {
     enz: true,
+    xhr_calls: true,
   },
   plugins: [
     'react'
diff --git a/src/index.js b/src/index.js
index 3c18e926..f5fb49ba 100644
--- a/src/index.js
+++ b/src/index.js
@@ -154,6 +154,7 @@ window.socket = function (public_key, config) {
       } else {
         router.history.push(url_generator(path))
       }
-    }
+    },
+    config: config,
   }
 }
diff --git a/src/tests/App.test.js b/src/tests/App.test.js
index 07a74bb4..f098864c 100644
--- a/src/tests/App.test.js
+++ b/src/tests/App.test.js
@@ -1,10 +1,13 @@
 import React from 'react'
 import {BrowserRouter as Router} from 'react-router-dom'
 import App from '../components/App'
-import {tick} from './utils'
+import { xhr_setup, tick } from './utils'
+
+beforeEach(() => {
+  xhr_setup()
+})
 
 it('shows tutors', async () => {
-  // expect.assertions(2)
   const config = {
     router_mode: 'history',
     api_root: 'https://socket.tutorcruncher.com',
@@ -15,6 +18,34 @@ it('shows tutors', async () => {
   await tick()
   wrapper.update()
   // console.log(pretty_html(wrapper.html()))
-  expect(global.XMLHttpRequest.mock.calls.length).toBe(2)
+  expect(xhr_calls.length).toBe(2)
+  expect(xhr_calls[1]).toEqual({
+    method: 'GET',
+    url: 'https://socket.tutorcruncher.com/good/contractors',
+    args: null
+  })
+  expect(wrapper.find('.tcs-col').length).toBe(2)
+})
+
+it('with con filter', async () => {
+  const config = {
+    router_mode: 'history',
+    api_root: 'https://socket.tutorcruncher.com',
+    mode: 'grid',
+    contractor_filter: {
+      label: ['foobar'],
+      label_exclude: ['spam'],
+    },
+    event_callback: () => null,
+  }
+  const wrapper = enz.mount(<Router><App config={config} public_key={'good'} url_generator={u => u}/></Router>)
+  await tick()
+  wrapper.update()
+  expect(xhr_calls.length).toBe(2)
+  expect(xhr_calls[1]).toEqual({
+    method: 'GET',
+    url: 'https://socket.tutorcruncher.com/good/contractors',
+    args: 'label=foobar&label_exclude=spam'
+  })
   expect(wrapper.find('.tcs-col').length).toBe(2)
 })
diff --git a/src/tests/index.test.js b/src/tests/index.test.js
index 73bd571e..1a24e34a 100644
--- a/src/tests/index.test.js
+++ b/src/tests/index.test.js
@@ -1,12 +1,37 @@
 import '../index'
+import { xhr_setup } from './utils'
+
+beforeEach(() => {
+  xhr_setup()
+})
 
 it('renders grid', () => {
   const div = document.createElement('div')
   div.setAttribute('id', 'socket')
   document.body.appendChild(div)
-  window.localStorage = {}
   const r = window.socket('good')
   expect(r.goto).toBeTruthy()
+  expect(r.config.contractor_filter).toEqual({})
   expect(div.querySelectorAll('.tcs-grid').length).toBe(1)
   // console.log(pretty_html(div.innerHTML))
 })
+
+
+it('converts contractor filter', () => {
+  const div = document.createElement('div')
+  div.setAttribute('id', 'socket')
+  document.body.appendChild(div)
+  const r = window.socket('good', {
+    // router_mode: 'history',
+    mode: 'grid',
+    labels_include: ['foobar'],
+    labels_exclude: ['spam'],
+    event_callback: () => null,
+  })
+  expect(r.goto).toBeTruthy()
+  expect(r.config.contractor_filter).toEqual({
+    'label': ['foobar'],
+    'label_exclude': ['spam'],
+  })
+  // console.log(pretty_html(div.innerHTML))
+})
diff --git a/src/tests/setup.js b/src/tests/setup.js
index e87e5ad3..0a6cc4f1 100644
--- a/src/tests/setup.js
+++ b/src/tests/setup.js
@@ -3,13 +3,11 @@ import {configure} from 'enzyme'
 import Adapter from 'enzyme-adapter-react-16'
 import {shallow, mount, render} from 'enzyme'
 import pretty from 'pretty'
-import {MockXMLHttpRequest} from './utils'
-
 
 configure({
   adapter: new Adapter()
 })
-global.XMLHttpRequest = jest.fn(MockXMLHttpRequest)
+window.localStorage = {}
 global.pretty_html = pretty
 global.enz = {
   shallow: shallow,
diff --git a/src/tests/utils.js b/src/tests/utils.js
index 299dcb1c..784c6ddc 100644
--- a/src/tests/utils.js
+++ b/src/tests/utils.js
@@ -1,13 +1,13 @@
 const RESPONSES = {
-  'GET:https://socket.tutorcruncher.com/good/subjects': {
+  'GET:https://socket.tutorcruncher.com/good/subjects': () => ({
     status: 200,
     content: JSON.stringify([
       {'id': 29, 'name': 'English Language', 'category': 'English', 'link': '29-english-language'},
       {'id': 31, 'name': 'English Literature', 'category': 'English', 'link': '31-english-literature'},
       {'id': 61, 'name': 'Chinese', 'category': 'Languages', 'link': '61-chinese'}
     ])
-  },
-  'GET:https://socket.tutorcruncher.com/good/contractors': {
+  }),
+  'GET:https://socket.tutorcruncher.com/good/contractors': () => ({
     status: 200,
     content: JSON.stringify(
       [
@@ -36,7 +36,7 @@ const RESPONSES = {
           'distance': null
         }
       ])
-  }
+  })
 }
 
 export function MockXMLHttpRequest () {
@@ -57,9 +57,17 @@ export function MockXMLHttpRequest () {
   this.status = null
   this.statusText = null
   this.send = function (data) {
-    // console.log(`XHR ${_method}: ${_url}`)
-    const response = RESPONSES[`${_method}:${_url}`]
-    if (response) {
+    let args = null
+    if (_url.includes('?')) {
+      args = _url.substr(_url.indexOf('?') + 1, _url.length)
+      _url = _url.substr(0, _url.indexOf('?'))
+    }
+    // console.log(`XHR ${_method}: ${_url} args=${args}`)
+    const f = RESPONSES[`${_method}:${_url}`]
+    const req = {method: _method, url: _url, args}
+    global.xhr_calls.push(req)
+    if (f) {
+      const response = f(req)
       this.status = response.status
       this.responseText = response.content
       this.onload && this.onload()
@@ -70,5 +78,10 @@ export function MockXMLHttpRequest () {
 }
 
 export function tick () {
-  return new Promise(resolve => setTimeout(resolve, 0.01))
+  return new Promise(resolve => setTimeout(resolve, 0))
+}
+
+export function xhr_setup () {
+  global.xhr_calls = []
+  global.XMLHttpRequest = MockXMLHttpRequest
 }