JavaScriptSolidServer
diff --git a/‎__mocks__/rdflib.ts‎
Lines changed: 5 additions & 5 deletions b/‎__mocks__/rdflib.ts‎
Lines changed: 5 additions & 5 deletions
diff --git a/‎src/widgets/forms/basic.ts‎
Lines changed: 6 additions & 3 deletions b/‎src/widgets/forms/basic.ts‎
Lines changed: 6 additions & 3 deletions
diff --git a/‎test/unit/widgets/forms/__snapshots__/basic.test.ts.snap‎
Lines changed: 28 additions & 0 deletions b/‎test/unit/widgets/forms/__snapshots__/basic.test.ts.snap‎
Lines changed: 28 additions & 0 deletions
@@ -71,23 +71,23 @@ export class Fetcher {
 
 export class UpdateManager {
   // mock as needed
+  updated: boolean = false
+  reportSuccess: boolean = true
 
   editable (uri: string) {
-    if (uri === 'http://example.com/') {
-      return true
-    }
     if (uri === 'http://not.editable/') {
       return false
     }
-    console.error('please use only http://editable.com/ and http://not.editable/ as docs in test')
     return true
   }
 
   put () {
     return Promise.resolve()
   }
 
-  update () {
+  update (_deletes, _inserts, onDone: (uri: string, ok: boolean, body: string) => void) {
+    this.updated = true
+    onDone('uri', this.reportSuccess, 'body')
     return Promise.resolve()
   }
 }
@@ -24,6 +24,7 @@ export function fieldLabel (dom: HTMLDocument, property: NamedNode | undefined,
     return dom.createTextNode('@@Internal error: undefined property')
   }
   const anchor = dom.createElement('a')
+  /* istanbul ignore next */
   if (property.uri) anchor.setAttribute('href', property.uri)
   anchor.setAttribute('style', 'color: #3B5998; text-decoration: none;') // Not too blue and no underline
   anchor.textContent = lab
@@ -103,7 +104,7 @@ export function basicField (
   const uri = mostSpecificClassURI(form)
   let params = fieldParams[uri]
   if (params === undefined) params = {} // non-bottom field types can do this
-  const style = params.style || textInputStyle || 'font-size: 100%; margin: 0.1em; padding: 0.1em;'
+  const style = params.style || textInputStyle
   // box.appendChild(dom.createTextNode(' uri='+uri+', pattern='+ params.pattern))
   const field = dom.createElement('input')
   ;(field as any).style = textInputStyle // Do we have to override length etc?
@@ -129,6 +130,7 @@ export function basicField (
     field.value = decodeURIComponent(obj.uri.replace(params.uriPrefix, '')) // should have no spaces but in case
       .replace(/ /g, '')
   } else if (obj) {
+    /* istanbul ignore next */
     field.value = obj.value || obj.uri || ''
   }
   field.setAttribute('style', style)
@@ -178,7 +180,7 @@ export function basicField (
           result = new Literal(field.value)
         }
       }
-      let is = ds.map(st => st(st.subject, st.predicate, result, st.why)) // can include >1 doc
+      let is = ds.map(statement => st(statement.subject, statement.predicate, result, statement.why)) // can include >1 doc
       if (is.length === 0) {
         // or none
         is = [st(subject, property, result, doc as Node)]
@@ -190,8 +192,10 @@ export function basicField (
           if (!docs.includes(st.why.uri)) docs.push(st.why.uri)
         })
         ds.forEach(st => {
+          /* istanbul ignore next */
           if (!docs.includes(st.why.uri)) docs.push(st.why.uri)
         })
+        /* istanbul ignore next */
         if (docs.length === 0) {
           throw new Error('updateMany has no docs to patch')
         }
@@ -209,7 +213,6 @@ export function basicField (
           if (ok) {
             updateMany(ds2, is2, callback)
           } else {
-            console.log('Update many failed on: ' + doc)
             callback(uri, ok, body)
           }
         })
 
@@ -1,5 +1,33 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
+exports[`basicField adds the basic field to the container, if provided 1`] = `
+<div>
+  <tr>
+    <td
+      class="formFieldName"
+      style="  vertical-align: middle;"
+    >
+      <a
+        href="http://example.com/#some-property"
+        style="color: #3B5998; text-decoration: none;"
+      >
+        Some property
+      </a>
+    </td>
+    <td
+      class="formFieldValue"
+    >
+      <input
+        maxlength="4096"
+        size="20"
+        style="background-color: #eef; padding: 0.5em;  border: .05em solid #88c;  border-radius:0.2em; font-size: 100%; margin:0.2em; "
+        type="text"
+      />
+    </td>
+  </tr>
+</div>
+`;
+
 exports[`basicField creates a basic field 1`] = `
 <tr>
   <td
Original file line number	Diff line number	Diff line change
`@@ -71,23 +71,23 @@ export class Fetcher {`
`71`	`71`
`72`	`72`	`export class UpdateManager {`
`73`	`73`	`// mock as needed`
	`74`	`+ updated: boolean = false`
	`75`	`+ reportSuccess: boolean = true`
`74`	`76`
`75`	`77`	`editable (uri: string) {`
`76`		`- if (uri === 'http://example.com/') {`
`77`		`- return true`
`78`		`- }`
`79`	`78`	`if (uri === 'http://not.editable/') {`
`80`	`79`	`return false`
`81`	`80`	`}`
`82`		`- console.error('please use only http://editable.com/ and http://not.editable/ as docs in test')`
`83`	`81`	`return true`
`84`	`82`	`}`
`85`	`83`
`86`	`84`	`put () {`
`87`	`85`	`return Promise.resolve()`
`88`	`86`	`}`
`89`	`87`
`90`		`- update () {`
	`88`	`+ update (_deletes, _inserts, onDone: (uri: string, ok: boolean, body: string) => void) {`
	`89`	`+ this.updated = true`
	`90`	`+ onDone('uri', this.reportSuccess, 'body')`
`91`	`91`	`return Promise.resolve()`
`92`	`92`	`}`
`93`	`93`	`}`