Changes while reviewing and testing

Author: thehabes

app.js

@@ -59,7 +59,6 @@ app.use(
 app.use(logger('dev'))
 app.use(express.json({ type: ["application/json", "application/ld+json"] }))
 app.use(express.text())
-app.use(express.urlencoded({ extended: true }))
 app.use(cookieParser())
 
 //Publicly available scripts, CSS, and HTML pages.

routes/__tests__/bulkCreate.test.js

@@ -13,7 +13,6 @@ const addAuth = (req, res, next) => {
 
 const routeTester = new express()
 routeTester.use(express.json())
-routeTester.use(express.urlencoded({ extended: false }))
 
 // Mount our own /bulkCreate route without auth that will use controller.bulkCreate
 routeTester.use("/bulkCreate", [addAuth, controller.bulkCreate])

routes/__tests__/bulkUpdate.test.js

@@ -13,7 +13,6 @@ const addAuth = (req, res, next) => {
 
 const routeTester = new express()
 routeTester.use(express.json())
-routeTester.use(express.urlencoded({ extended: false }))
 
 // Mount our own /bulkCreate route without auth that will use controller.bulkCreate
 routeTester.use("/bulkUpdate", [addAuth, controller.bulkUpdate])

routes/__tests__/contentType.test.js

@@ -6,7 +6,7 @@ import rest from '../../rest.js'
 const routeTester = express()
 routeTester.use(express.json({ type: ["application/json", "application/ld+json"] }))
 routeTester.use(express.text())
-routeTester.use(express.urlencoded({ extended: true }))
+
 
 // Mount the validateContentType middleware on /api just like api-routes.js
 routeTester.use("/api", rest.validateContentType)

routes/__tests__/create.test.js

@@ -14,7 +14,6 @@ const addAuth = (req, res, next) => {
 
 const routeTester = new express()
 routeTester.use(express.json())
-routeTester.use(express.urlencoded({ extended: false }))
 
 // Mount our own /create route without auth that will use controller.create
 routeTester.use("/create", [addAuth, controller.create])

routes/__tests__/delete.test.js

@@ -13,7 +13,6 @@ const addAuth = (req, res, next) => {
 
 const routeTester = new express()
 routeTester.use(express.json())
-routeTester.use(express.urlencoded({ extended: false }))
 
 // FIXME here we need to create something to delete in order to test this route.
 routeTester.use("/create", [addAuth, controller.create])

routes/__tests__/history.test.js

@@ -8,7 +8,6 @@ import controller from '../../db-controller.js'
 
 const routeTester = new express()
 routeTester.use(express.json())
-routeTester.use(express.urlencoded({ extended: false }))
 
 // Mount our own /history route without auth that will use controller.history
 routeTester.use("/history/:_id", controller.history)

routes/__tests__/id.test.js

@@ -7,7 +7,6 @@ import controller from '../../db-controller.js'
 
 const routeTester = new express()
 routeTester.use(express.json())
-routeTester.use(express.urlencoded({ extended: false }))
 
 // Mount our own /id route without auth that will use controller.id
 routeTester.use("/id/:_id", controller.id)

routes/__tests__/overwrite-optimistic-locking.test.txt

@@ -26,7 +26,6 @@ const addAuth = (req, res, next) => {
 // Create a test Express app
 const routeTester = express()
 routeTester.use(express.json())
-routeTester.use(express.urlencoded({ extended: false }))
 
 // Mount our routes
 routeTester.use('/overwrite', [addAuth, controller.overwrite])

routes/__tests__/patch.test.js

@@ -14,7 +14,6 @@ const addAuth = (req, res, next) => {
 
 const routeTester = new express()
 routeTester.use(express.json())
-routeTester.use(express.urlencoded({ extended: false }))
 
 // Mount our own /patch route without auth that will use controller.patch
 routeTester.use("/patch", [addAuth, controller.patchUpdate])