Merge pull request #6707 from LibreSign/backport/6705/stable33

[stable33] fix: add able to sign getter

Author: vitormattos

src/store/sign.js

@@ -13,6 +13,7 @@ import axios from '@nextcloud/axios'
 import { useFilesStore } from './files.js'
 import { useSidebarStore } from './sidebar.js'
 import { useSignMethodsStore } from './signMethods.js'
+import { FILE_STATUS } from '../constants.js'
 
 const defaultState = {
 	errors: [],
@@ -36,6 +37,26 @@ const defaultState = {
 export const useSignStore = defineStore('sign', {
 	state: () => ({ ...defaultState }),
 
+	getters: {
+		ableToSign(state) {
+			const allowedStatuses = [FILE_STATUS.ABLE_TO_SIGN, FILE_STATUS.PARTIAL_SIGNED]
+			if (!allowedStatuses.includes(state.document?.status)) {
+				return false
+			}
+
+			const mySigner = state.document?.signers?.find(signer => signer.me)
+			if (!mySigner) {
+				return false
+			}
+
+			if (mySigner.status !== 1) {
+				return false
+			}
+
+			return true
+		},
+	},
+
 	actions: {
 		async initFromState() {
 			this.errors = loadState('libresign', 'errors', [])

src/store/sign.spec.js

@@ -6,6 +6,7 @@
 import { describe, expect, it, beforeEach } from 'vitest'
 import { setActivePinia, createPinia } from 'pinia'
 import { useSignStore } from './sign.js'
+import { FILE_STATUS } from '../constants.js'
 
 describe('useSignStore', () => {
 	beforeEach(() => {
@@ -44,4 +45,66 @@ describe('useSignStore', () => {
 			expect(store.pendingAction).toBe('createPassword')
 		})
 	})
+
+	describe('ableToSign getter', () => {
+		it('returns false when document status is not ABLE_TO_SIGN or PARTIAL_SIGNED', () => {
+			const store = useSignStore()
+			store.document = {
+				status: FILE_STATUS.DRAFT,
+				signers: [{ me: true, status: 1 }],
+			}
+			expect(store.ableToSign).toBe(false)
+		})
+
+		it('returns false when there is no signer with me: true', () => {
+			const store = useSignStore()
+			store.document = {
+				status: FILE_STATUS.ABLE_TO_SIGN,
+				signers: [{ me: false, status: 1 }],
+			}
+			expect(store.ableToSign).toBe(false)
+		})
+
+		it('returns false when signer status is not ABLE_TO_SIGN (status 1)', () => {
+			const store = useSignStore()
+			store.document = {
+				status: FILE_STATUS.ABLE_TO_SIGN,
+				signers: [{ me: true, status: 0 }],
+			}
+			expect(store.ableToSign).toBe(false)
+		})
+
+		it('returns true when document status is ABLE_TO_SIGN and signer can sign', () => {
+			const store = useSignStore()
+			store.document = {
+				status: FILE_STATUS.ABLE_TO_SIGN,
+				signers: [{ me: true, status: 1 }],
+			}
+			expect(store.ableToSign).toBe(true)
+		})
+
+		it('returns true when document status is PARTIAL_SIGNED and signer can sign', () => {
+			const store = useSignStore()
+			store.document = {
+				status: FILE_STATUS.PARTIAL_SIGNED,
+				signers: [{ me: true, status: 1 }],
+			}
+			expect(store.ableToSign).toBe(true)
+		})
+
+		it('returns false when document is undefined', () => {
+			const store = useSignStore()
+			store.document = undefined
+			expect(store.ableToSign).toBe(false)
+		})
+
+		it('returns false when signers array is empty', () => {
+			const store = useSignStore()
+			store.document = {
+				status: FILE_STATUS.ABLE_TO_SIGN,
+				signers: [],
+			}
+			expect(store.ableToSign).toBe(false)
+		})
+	})
 })