Browse Source

Merge branch 'master' of github.com:Budibase/budibase into fix/roles-issue

pull/1605/head
mike12345567 5 years ago
parent
commit
0382939fa5
  1. 6
      packages/server/src/api/controllers/datasource.js
  2. 2
      packages/server/src/middleware/tests/authorized.spec.js
  3. 2
      packages/server/src/middleware/tests/resourceId.spec.js

6
packages/server/src/api/controllers/datasource.js

@ -30,9 +30,9 @@ exports.save = async function (ctx) {
datasource._rev = response.rev
// Drain connection pools when configuration is changed
const pool = integrations[datasource.source].pool
if (pool) {
await pool.end()
const source = integrations[datasource.source]
if (source && source.pool) {
await source.pool.end()
}
ctx.status = 200

2
packages/server/src/middleware/tests/authorized.spec.js

@ -5,7 +5,7 @@ jest.mock("../../environment", () => ({
prod: false,
isTest: () => true,
isProd: () => this.prod,
_set: (key, value) => {
_set: function(key, value) {
this.prod = value === "production"
}
})

2
packages/server/src/middleware/tests/resourceId.spec.js

@ -91,7 +91,7 @@ describe("resourceId middleware", () => {
.mainResource("custom")
.subResource("customSub")
.build()
config = new TestConfiguration(middleware)
let config = new TestConfiguration(middleware)
config.setBody({
custom: "test",
customSub: "subtest"

Loading…
Cancel
Save