author.uri and author.email should be optional in atom #115 (#124)

* email/uri are optional for atom authors

* test: add coverage for optional cases

* test: update syntax

Co-authored-by: Josh Santangelo <josh@endquote.com>
Co-authored-by: Decebal Dobrica <decebal.dobrica@tellimer.com>

src/__tests__/__snapshots__/atom1.spec.ts.snap

@@ -42,6 +42,9 @@ exports[`atom 1.0 should generate a valid feed 1`] = `
             <email>joesmith@example.com</email>
             <uri>https://example.com/joesmith</uri>
         </author>
+        <author>
+            <name>Joe Smith, Name Only</name>
+        </author>
         <category label=\\"Grateful Dead\\"/>
         <category label=\\"MSFT\\"/>
         <contributor>

src/__tests__/setup.ts

@@ -45,13 +45,16 @@ sampleFeed.addItem({
     {
       name: "Jane Doe",
       email: "janedoe@example.com",
-      link: "https://example.com/janedoe?link=sanitized&value=2"
+      link: "https://example.com/janedoe?link=sanitized&value=2",
     },
     {
       name: "Joe Smith",
       email: "joesmith@example.com",
       link: "https://example.com/joesmith",
     },
+    {
+      name: "Joe Smith, Name Only",
+    }
   ],
   contributor: [
     {

src/atom1.ts

@@ -167,21 +167,26 @@ export default (ins: Feed) => {
 };
 
 /**
- * Returns a formated author
+ * Returns a formatted author
  * @param author
  */
 const formatAuthor = (author: Author) => {
   const { name, email, link } = author;
 
-  return {
-    name,
-    email,
-    uri: sanitize(link)
-  };
+  const out: { name?: string, email?: string, uri?: string } = { name };
+  if (email) {
+    out.email = email;
+  }
+
+  if (link) {
+    out.uri = sanitize(link);
+  }
+
+  return out;
 };
 
 /**
- * Returns a formated category
+ * Returns a formatted category
  * @param category
  */
 const formatCategory = (category: Category) => {

src/utils.ts

@@ -1,4 +1,4 @@
-export function sanitize(url: String | undefined): String | undefined {
+export function sanitize(url: string | undefined): string | undefined {
   if (typeof (url) === 'undefined') {
     return;
   }