Removed comments added skips instead also color comment
DELETE
TestTracking.xlsx
TestTracking.xlsx
Binary files a/TestTracking.xlsx and /dev/null differ

MODIFY
test/unit/pad.test.ts
test/unit/pad.test.ts
@@ -2,7 +2,6 @@ import { JSDOM } from 'jsdom'
import * as RdfLib from 'rdflib'
import * as pad from '../../src/pad'
const widgets = require('../../src/widgets')
-const store = require('../../src/store')
jest.mock('rdflib')
jest.mock('solid-auth-client')
@@ -13,6 +12,8 @@ describe('lightColorHash', () => {
it('exists', () => {
expect((pad as any).lightColorHash).toBeInstanceOf(Function)
})
+ // #ffffff is specifically stated in the code to be returned
+ // when there is no author
it('returns #ffffff when an author is not provided', () => {
expect((pad as any).lightColorHash(null)).toBe('#ffffff')
})
@@ -44,7 +45,7 @@ describe('renderPartipants', () => {
// kb is a store
const table = dom.createElement('table')
const padDoc = null
- debugger
+
const subject = RdfLib.sym('participation')
const me = 'webId'
const options = {}
@@ -140,8 +141,8 @@ describe('notepad', () => {
/>
`)
})
- /*
- it('should log error that you need to be logged in for pad to be edited', () => {
+
+ it.skip('should log error that you need to be logged in for pad to be edited', () => {
const padDoc = null
const subject = null
const me = null
@@ -156,9 +157,9 @@ describe('notepad', () => {
expect(console.log).toBeCalledWith(
'Warning: must be logged in for pad to be edited'
)
- }) */
- /*
- it('status area ...', () => {
+ })
+
+ it.skip('status area ...', () => {
const padDoc = null
const subject = null
const me = { uri: 'https://sharonstrats.inrupt.net/profile/card#me' }
@@ -168,11 +169,11 @@ describe('notepad', () => {
(pad as any).notepad(dom, padDoc, subject, me, options)
).resolves.toMatchInlineSnapshot()
})
- */
+
// @@ TODO the code itself seems to error where it says 'new'
// need to research further
- /*
- it('should throw an error when me is provided but no uri', () => {
+
+ it.skip('should throw an error when me is provided but no uri', () => {
const padDoc = null
const subject = null
const me = {}
@@ -181,5 +182,5 @@ describe('notepad', () => {
expect(
(pad as any).notepad(dom, padDoc, subject, me, options)
).resolves.toBe({})
- }) */
+ })
})

MODIFY
test/unit/widgets/forms.test.ts
test/unit/widgets/forms.test.ts
@@ -37,8 +37,8 @@ describe('Form field', () => {
it('exists', () => {
expect(field[ns.ui('Form').uri]).toBeInstanceOf(Object)
})
- /*
- it('runs', () => {
+
+ it.skip('runs', () => {
const container = document.createElement('div')
const already = {}
const subject = RdfLib.sym('')
@@ -79,7 +79,7 @@ describe('Form field', () => {
})
// @@ TODO check this further what test could I use to make sure
// to test that the form gets added but obviously not in the container passed in
- /* it('still returns an element if the container is null', () => {
+ it.skip('still returns an element if the container is null', () => {
// debugger
const container = null
const already = {}
@@ -106,9 +106,9 @@ describe('Form field', () => {
</h3>
</div>
`)
- }) */
+ })
// @@ TODO need to double check proper subjects and what gets returned
- /* it('returns.. if the subject has already been processed...', () => {
+ it.skip('returns.. if the subject has already been processed...', () => {
const dubSubject = RdfLib.sym('subject')
const container = document.createElement('container')
const already = { dubSubject }
@@ -135,7 +135,7 @@ describe('Form field', () => {
</h3>
</div>
`)
- }) */
+ })
})
describe('Options field', () => {
@@ -167,8 +167,8 @@ describe('Multiple field', () => {
it('exists', () => {
expect(field[ns.ui('Multiple').uri]).toBeInstanceOf(Object)
})
- /*
- it('runs', () => {
+
+ it.skip('runs', () => {
const container = document.createElement('div')
const already = {}
const subject = RdfLib.sym('')
@@ -194,7 +194,7 @@ describe('Multiple field', () => {
</h3>
</div>
`)
- }) */
+ })
})
describe('fieldParams', () => {
@@ -1060,8 +1060,8 @@ describe('Choice', () => {
it('exists', () => {
expect(field[ns.ui('Choice').uri]).toBeInstanceOf(Object)
})
- /*
- it('runs', () => {
+
+ it.skip('runs', () => {
const container = document.createElement('div')
const already = {}
const subject = RdfLib.sym('')
@@ -1087,7 +1087,7 @@ describe('Choice', () => {
</h3>
</div>
`)
- }) */
+ })
})
describe('Comment params', () => {
@@ -1426,10 +1426,10 @@ describe('newThing', () => {
it('runs', () => {
expect(newThing(RdfLib.sym(''))).toBeInstanceOf(Object)
})
- /* need to also mock or figure out doc - which is a NamedNode
- it('returns the correct .', () => {
+ // need to also mock or figure out doc - which is a NamedNode
+ it.skip('returns the correct .', () => {
const Date = jest.fn()
Date.mockReturnValueOnce('Thu Feb 06 2020 19:42:59 GMT+1100')
expect(newThing('doc').toBe(''))
- }) */
+ })
})

MODIFY
test/unit/widgets/peoplePicker.test.ts
test/unit/widgets/peoplePicker.test.ts
@@ -66,7 +66,6 @@ describe('PeoplePicker.render', () => {
groupPickedCb,
options
)
- debugger
peoplePicker.render()
expect(spy).toHaveBeenCalledTimes(2)
})
@@ -81,8 +80,7 @@ describe('PeoplePicker.render', () => {
groupPickedCb,
options
)
- // debugger
- // peoplePicker.render()
+
expect(peoplePicker.render()).toMatchInlineSnapshot(`
PeoplePicker {
"element": <p>

Removed comments added skips instead also color comment