Merge branch 'master' into redux-toolkit-tree-view

Backend.Tests/Controllers/LiftControllerTests.cs

@@ -225,6 +225,38 @@ private static async Task<string> DownloadAndReadLift(LiftController liftControl
             return liftText;
         }
 
+        [Test]
+        public void TestUploadLiftFileNoPermission()
+        {
+            _liftController.ControllerContext.HttpContext = PermissionServiceMock.UnauthorizedHttpContext();
+            var result = _liftController.UploadLiftFile(_projId, new FileUpload()).Result;
+            Assert.That(result, Is.InstanceOf<ForbidResult>());
+        }
+
+        [Test]
+        public void TestUploadLiftFileInvalidProjectId()
+        {
+            var result = _liftController.UploadLiftFile("../hack", new FileUpload()).Result;
+            Assert.That(result, Is.InstanceOf<UnsupportedMediaTypeResult>());
+        }
+
+        [Test]
+        public void TestUploadLiftFileAlreadyImported()
+        {
+            var projId = _projRepo.Create(new Project { Name = "already has import", LiftImported = true }).Result!.Id;
+            var result = _liftController.UploadLiftFile(projId, new FileUpload()).Result;
+            Assert.That(result, Is.InstanceOf<BadRequestObjectResult>());
+            Assert.That(((BadRequestObjectResult)result).Value, Contains.Substring("LIFT"));
+        }
+
+        [Test]
+        public void TestUploadLiftFileBadFile()
+        {
+            var result = _liftController.UploadLiftFile(_projId, new FileUpload()).Result;
+            Assert.That(result, Is.InstanceOf<BadRequestObjectResult>());
+            Assert.That(((BadRequestObjectResult)result).Value, Is.InstanceOf<string>());
+        }
+
         [Test]
         public void TestUploadLiftFileAndGetWritingSystems()
         {
@@ -271,6 +303,21 @@ public void TestFinishUploadLiftFileNothingToFinish()
             Assert.That(_liftService.RetrieveImport(UserId), Is.Null);
         }
 
+        [Test]
+        public void TestFinishUploadLiftFileNoPermission()
+        {
+            _liftController.ControllerContext.HttpContext = PermissionServiceMock.UnauthorizedHttpContext();
+            var result = _liftController.FinishUploadLiftFile(_projId).Result;
+            Assert.That(result, Is.InstanceOf<ForbidResult>());
+        }
+
+        [Test]
+        public void TestFinishUploadLiftFileInvalidProjectId()
+        {
+            var result = _liftController.FinishUploadLiftFile("../hack", UserId).Result;
+            Assert.That(result, Is.InstanceOf<UnsupportedMediaTypeResult>());
+        }
+
         [Test]
         public async Task TestModifiedTimeExportsToLift()
         {
@@ -285,6 +332,35 @@ public async Task TestModifiedTimeExportsToLift()
             Assert.That(liftContents, Does.Contain("dateModified=\"2000-01-01T00:00:00Z\""));
         }
 
+        [Test]
+        public void TestExportLiftFileNoPermission()
+        {
+            _liftController.ControllerContext.HttpContext = PermissionServiceMock.UnauthorizedHttpContext();
+            var result = _liftController.ExportLiftFile(_projId).Result;
+            Assert.That(result, Is.InstanceOf<ForbidResult>());
+        }
+
+        [Test]
+        public void TestExportLiftFileInvalidProjectId()
+        {
+            var result = _liftController.ExportLiftFile("../hack").Result;
+            Assert.That(result, Is.InstanceOf<UnsupportedMediaTypeResult>());
+        }
+
+        [Test]
+        public void TestExportLiftFileNoProject()
+        {
+            var result = _liftController.ExportLiftFile("non-existent-project").Result;
+            Assert.That(result, Is.InstanceOf<NotFoundObjectResult>());
+        }
+
+        [Test]
+        public void TestExportLiftFileNoWordsInProject()
+        {
+            var result = _liftController.ExportLiftFile(_projId).Result;
+            Assert.That(result, Is.InstanceOf<BadRequestObjectResult>());
+        }
+
         [Test]
         public void TestExportInvalidProjectId()
         {
@@ -294,6 +370,47 @@ public void TestExportInvalidProjectId()
                 Throws.TypeOf<MissingProjectException>());
         }
 
+        [Test]
+        public void TestDownloadLiftFileNoPermission()
+        {
+            _liftController.ControllerContext.HttpContext = PermissionServiceMock.UnauthorizedHttpContext();
+            var result = _liftController.DownloadLiftFile(_projId).Result;
+            Assert.That(result, Is.InstanceOf<ForbidResult>());
+        }
+
+        [Test]
+        public void TestCanUploadLiftNoPermission()
+        {
+            _liftController.ControllerContext.HttpContext = PermissionServiceMock.UnauthorizedHttpContext();
+            var result = _liftController.CanUploadLift(_projId).Result;
+            Assert.That(result, Is.InstanceOf<ForbidResult>());
+        }
+
+        [Test]
+        public void TestCanUploadLiftInvalidProjectId()
+        {
+            var result = _liftController.CanUploadLift("../hack").Result;
+            Assert.That(result, Is.InstanceOf<UnsupportedMediaTypeResult>());
+        }
+
+        [Test]
+        public void TestCanUploadLiftFalse()
+        {
+            var projId = _projRepo.Create(new Project { Name = "has import", LiftImported = true }).Result!.Id;
+            var result = _liftController.CanUploadLift(projId).Result;
+            Assert.That(result, Is.InstanceOf<OkObjectResult>());
+            Assert.That(((OkObjectResult)result).Value, Is.False);
+        }
+
+        [Test]
+        public void TestCanUploadLiftTrue()
+        {
+            var projId = _projRepo.Create(new Project { Name = "has no import", LiftImported = false }).Result!.Id;
+            var result = _liftController.CanUploadLift(projId).Result;
+            Assert.That(result, Is.InstanceOf<OkObjectResult>());
+            Assert.That(((OkObjectResult)result).Value, Is.True);
+        }
+
         /// <summary>
         /// Create three words and delete one. Ensure that the deleted word is still exported to Lift format and marked
         /// as deleted.

Backend.Tests/Mocks/ProjectRepositoryMock.cs

@@ -95,7 +95,8 @@ public Task<ResultOfUpdate> Update(string projectId, Project project)
 
         public Task<bool> CanImportLift(string projectId)
         {
-            return Task.FromResult(true);
+            var project = _projects.Find(p => p.Id == projectId);
+            return Task.FromResult(project?.LiftImported != true);
         }
     }
 }

Backend/Services/LiftService.cs

@@ -1,5 +1,6 @@
 using System;
 using System.Collections.Generic;
+using System.Diagnostics.CodeAnalysis;
 using System.IO;
 using System.IO.Compression;
 using System.Linq;
@@ -944,16 +945,17 @@ public void ProcessRangeElement(string range, string id, string guid, string par
 
             // The following are unused and are not implemented, but may still be called by the Lexicon Merger
             // They may be useful later if we need to add more complex attributes to words in The Combine
+            [ExcludeFromCodeCoverage]
             public LiftExample GetOrMakeExample(LiftSense sense, Extensible info)
             {
                 return new LiftExample { Content = new LiftMultiText() };
             }
-
+            [ExcludeFromCodeCoverage]
             public LiftObject GetOrMakeParentReversal(LiftObject parent, LiftMultiText contents, string type)
             {
                 return new LiftReversal();
             }
-
+            [ExcludeFromCodeCoverage]
             public LiftSense GetOrMakeSubsense(LiftSense sense, Extensible info, string rawXml)
             {
                 return new LiftSense(info, new Guid(), sense)
@@ -962,35 +964,40 @@ public LiftSense GetOrMakeSubsense(LiftSense sense, Extensible info, string rawX
                     Gloss = new LiftMultiText()
                 };
             }
-
+            [ExcludeFromCodeCoverage]
             public LiftObject MergeInEtymology(LiftEntry entry, string source, string type, LiftMultiText form,
-                LiftMultiText gloss, string rawXml)
+                        LiftMultiText gloss, string rawXml)
             {
                 return new LiftEtymology();
             }
-
+            [ExcludeFromCodeCoverage]
             public LiftObject MergeInReversal(
-                LiftSense sense, LiftObject parent, LiftMultiText contents, string type, string rawXml)
+                        LiftSense sense, LiftObject parent, LiftMultiText contents, string type, string rawXml)
             {
                 return new LiftReversal();
             }
-
+            [ExcludeFromCodeCoverage]
             public LiftObject MergeInVariant(LiftEntry entry, LiftMultiText contents, string rawXml)
             {
                 return new LiftVariant();
             }
-
+            [ExcludeFromCodeCoverage]
             public void EntryWasDeleted(Extensible info, DateTime dateDeleted) { }
+            [ExcludeFromCodeCoverage]
             public void MergeInExampleForm(LiftExample example, LiftMultiText multiText) { }
-
+            [ExcludeFromCodeCoverage]
             public void MergeInPicture(LiftSense sense, string href, LiftMultiText caption) { }
+            [ExcludeFromCodeCoverage]
             public void MergeInRelation(
-                LiftObject extensible, string relationTypeName, string targetId, string rawXml)
+                        LiftObject extensible, string relationTypeName, string targetId, string rawXml)
             { }
+            [ExcludeFromCodeCoverage]
             public void MergeInSource(LiftExample example, string source) { }
+            [ExcludeFromCodeCoverage]
             public void MergeInTranslationForm(
-                LiftExample example, string type, LiftMultiText multiText, string rawXml)
+                        LiftExample example, string type, LiftMultiText multiText, string rawXml)
             { }
+            [ExcludeFromCodeCoverage]
             public void ProcessFieldDefinition(string tag, LiftMultiText description) { }
         }
     }

src/components/DataEntry/DataEntryTable/NewEntry/SenseDialog.tsx

@@ -1,8 +1,9 @@
+import { Close } from "@mui/icons-material";
 import {
   Dialog,
   DialogContent,
-  Divider,
   Grid,
+  IconButton,
   MenuList,
   Typography,
 } from "@mui/material";
@@ -50,7 +51,7 @@ export default function SenseDialog(props: SenseDialogProps): ReactElement {
 
 interface SenseListProps {
   selectedWord: Word;
-  closeDialog: (gloss: string) => void;
+  closeDialog: (gloss?: string) => void;
   analysisLang: string;
 }
 
@@ -95,11 +96,7 @@ export function SenseList(props: SenseListProps): ReactElement {
     );
   };
 
-  const menuItems: ReactElement[] = [];
-  for (const s of props.selectedWord.senses) {
-    menuItems.push(menuItem(s));
-    menuItems.push(<Divider key={`${s.guid}-divider`} />);
-  }
+  const menuItems = props.selectedWord.senses.map(menuItem);
   menuItems.push(
     <StyledMenuItem key="new-sense" onClick={() => props.closeDialog("")}>
       {t("addWords.newSenseFor")}
@@ -109,7 +106,16 @@ export function SenseList(props: SenseListProps): ReactElement {
 
   return (
     <>
+      {/* Cancel button */}
+      <IconButton
+        onClick={() => props.closeDialog()}
+        style={{ position: "absolute", right: 0, top: 0 }}
+      >
+        <Close />
+      </IconButton>
+      {/* Header */}
       <Typography variant="h3">{t("addWords.selectSense")}</Typography>
+      {/* Sense options */}
       <MenuList autoFocusItem>{menuItems}</MenuList>
     </>
   );

src/components/DataEntry/DataEntryTable/NewEntry/StyledMenuItem.ts

@@ -4,6 +4,9 @@ import { withStyles } from "@mui/styles";
 // Copied from customized menus at https://material-ui.com/components/menus/
 const StyledMenuItem = withStyles((theme) => ({
   root: {
+    border: "1px solid gray",
+    borderRadius: "8px",
+    marginTop: "8px",
     "&:focus": {
       backgroundColor: theme.palette.primary.main,
       "& .MuiListItemIcon-root, & .MuiListItemText-primary": {

src/components/DataEntry/DataEntryTable/NewEntry/VernDialog.tsx

@@ -1,8 +1,9 @@
+import { Close } from "@mui/icons-material";
 import {
   Dialog,
   DialogContent,
-  Divider,
   Grid,
+  IconButton,
   MenuList,
   Typography,
 } from "@mui/material";
@@ -50,7 +51,7 @@ export default function VernDialog(props: vernDialogProps): ReactElement {
 
 interface VernListProps {
   vernacularWords: Word[];
-  closeDialog: (wordId: string) => void;
+  closeDialog: (wordId?: string) => void;
   analysisLang?: string;
 }
 
@@ -96,11 +97,7 @@ export function VernList(props: VernListProps): ReactElement {
     );
   };
 
-  const menuItems: ReactElement[] = [];
-  for (const w of props.vernacularWords) {
-    menuItems.push(menuItem(w));
-    menuItems.push(<Divider key={`${w.id}-divider`} />);
-  }
+  const menuItems = props.vernacularWords.map(menuItem);
   menuItems.push(
     <StyledMenuItem key="new-entry" onClick={() => props.closeDialog("")}>
       {t("addWords.newEntryFor")}
@@ -110,7 +107,16 @@ export function VernList(props: VernListProps): ReactElement {
 
   return (
     <>
+      {/* Cancel button */}
+      <IconButton
+        onClick={() => props.closeDialog()}
+        style={{ position: "absolute", right: 0, top: 0 }}
+      >
+        <Close />
+      </IconButton>
+      {/* Header */}
       <Typography variant="h3">{t("addWords.selectEntry")}</Typography>
+      {/* Entry options */}
       <MenuList autoFocusItem>{menuItems}</MenuList>
     </>
   );

src/components/Pronunciations/AudioPlayer.tsx

@@ -14,7 +14,7 @@ import { useTranslation } from "react-i18next";
 import { ButtonConfirmation } from "components/Dialogs";
 import {
   playing,
-  reset,
+  resetPronunciations,
 } from "components/Pronunciations/Redux/PronunciationsActions";
 import { PronunciationsStatus } from "components/Pronunciations/Redux/PronunciationsReduxTypes";
 import { StoreState } from "types";
@@ -38,8 +38,8 @@ const useStyles = makeStyles((theme: Theme) =>
 export default function AudioPlayer(props: PlayerProps): ReactElement {
   const isPlaying = useAppSelector(
     (state: StoreState) =>
-      state.pronunciationsState.payload === props.fileName &&
-      state.pronunciationsState.type === PronunciationsStatus.Playing
+      state.pronunciationsState.fileName === props.fileName &&
+      state.pronunciationsState.status === PronunciationsStatus.Playing
   );
 
   const [audio] = useState<HTMLAudioElement>(new Audio(props.pronunciationUrl));
@@ -48,7 +48,10 @@ export default function AudioPlayer(props: PlayerProps): ReactElement {
 
   const classes = useStyles();
   const dispatch = useAppDispatch();
-  const dispatchReset = useCallback(() => dispatch(reset()), [dispatch]);
+  const dispatchReset = useCallback(
+    () => dispatch(resetPronunciations()),
+    [dispatch]
+  );
   const { t } = useTranslation();
 
   useEffect(() => {