move_users #2

Closed
SergeyV wants to merge 0 commits from move_users into master
Collaborator

Updated users project

Updated users project
SergeyV added 2 commits 2024-04-02 11:42:54 +00:00
vbuglov approved these changes 2024-04-02 14:43:59 +00:00
@ -0,0 +49,4 @@
if (this.errorPassword) return
}
this.serviceOfUsers.editUserByService(params)
},
Owner

ревью для функции editData

функция
this.serviceOfUsers.editUserByService(params)
вызыается несколько раз. При чем если
this.errorPassword
значение будет 2 раза записано. Что делает editUserByService? Обновляет пользователя на сайте, записывает в стор? В таком случае хочется видеть префиксы update/set...комбинации слов в функции.
Может можно еще как - то сделать функцию читаемее

ревью для функции editData функция `this.serviceOfUsers.editUserByService(params)` вызыается несколько раз. При чем если `this.errorPassword` значение будет 2 раза записано. Что делает editUserByService? Обновляет пользователя на сайте, записывает в стор? В таком случае хочется видеть префиксы update/set...комбинации слов в функции. Может можно еще как - то сделать функцию читаемее
@ -0,0 +52,4 @@
},
saveUser() {
this.errorPassword = isRepeatPasswordEmpty(this.selectedUser.password, this.repeatPassword)
if (this.errorPassword) return
Owner

Такая запись путает, если if улетает на новую строку предланаю писать так:

 if (this.errorPassword) {
       return this.serviceOfUsers.saveUpdatedDataUser(this.selectedService.id)
 }

если это сброс писать так:

 if (this.errorPassword) return null
 this.serviceOfUsers.saveUpdatedDataUser(this.selectedService.id)
Такая запись путает, если if улетает на новую строку предланаю писать так: if (this.errorPassword) { return this.serviceOfUsers.saveUpdatedDataUser(this.selectedService.id) } если это сброс писать так: if (this.errorPassword) return null this.serviceOfUsers.saveUpdatedDataUser(this.selectedService.id)
@ -37,0 +144,4 @@
async saveUpdatedDataUser(serviceId) {
const selectedUser = this.store.getters['users/selectedUser']
// const userStatus = this.store.getters['users/userStatus']
if (selectedUser && selectedUser.id) {
Owner

Разнеси функцию

Разнеси функцию
@ -37,0 +166,4 @@
const users = this.store.getters['users/usersWithService']
const updatedUsers = updatedUser(selectedUser, users)
this.store.dispatch('users/updateUsersWithService', updatedUsers)
// return updatedUser
Owner

Удали комметарии которые не несут подсказок

Удали комметарии которые не несут подсказок
@ -17,0 +25,4 @@
async deleteUserFromService(userId) {
const selectedService = await this.serviceOfServices.getSelectedService()
if (!selectedService) {
console.error('removeUserFromService: Service not found')
Owner

logger.error

logger.error
@ -17,0 +29,4 @@
return {}
}
if (!userId) {
console.error('removeUserFromService: User not selected')
Owner

logger.error

logger.error
@ -18,0 +38,4 @@
async saveEditedData() {
const selectedService = await this.serviceOfServices.getSelectedService()
if (!selectedService) {
console.error('saveEditedData: Service not found')
Owner

logger.error

logger.error
@ -18,0 +55,4 @@
} else {
this.serviceOfServices.saveService(selectedService)
this.serviceOfUsers.saveUpdatedDataUser(selectedService.id)
}
Owner

разнеси функцию

разнеси функцию
@ -0,0 +33,4 @@
"internet_uri": "localhost",
"description": "localhost",
"is_online": true
},
Owner

Много где используется вынеси в mock файл, у меня были примеры
const defaultServices = [

Много где используется вынеси в mock файл, у меня были примеры const defaultServices = [
@ -0,0 +49,4 @@
"internet_uri": "localhost",
"description": "localhost",
"is_online": true
},
Owner

Много где используется вынеси в mock файл, у меня были примеры
const defaultServices = [

Много где используется вынеси в mock файл, у меня были примеры const defaultServices = [
@ -0,0 +71,4 @@
const wrapper = mount(EditServiceCard, {
global: {
plugins: [store]
Owner

In contrast, a unit test might isolate and test the component and the store separately. This can be useful if you have a very large application with a complex store. For this use case, you can mock the parts of the store you are interested in using global.mocks:

 const wrapper = mount(App, {
     global: {
       mocks: {
         $store
     }
    }
  })
In contrast, a unit test might isolate and test the component and the store separately. This can be useful if you have a very large application with a complex store. For this use case, you can mock the parts of the store you are interested in using global.mocks: const wrapper = mount(App, { global: { mocks: { $store } } })
@ -0,0 +97,4 @@
const wrapper = mount(EditServiceCard, {
global: {
plugins: [store]
Owner

const wrapper = mount(App, {
global: {
mocks: {
$store
}
}
})

const wrapper = mount(App, { global: { mocks: { $store } } })
@ -0,0 +143,4 @@
const wrapper = mount(EditServiceCard, {
global: {
plugins: [store]
Owner

const wrapper = mount(App, {
global: {
mocks: {
$store
}
}
})

const wrapper = mount(App, { global: { mocks: { $store } } })
@ -0,0 +36,4 @@
"internet_uri": "localhost",
"description": "localhost",
"is_online": true
},
Owner

Много где используется вынеси в mock файл, у меня были примеры
const defaultServices = [

...
посмотри везде

Много где используется вынеси в mock файл, у меня были примеры const defaultServices = [ ... посмотри везде
@ -0,0 +76,4 @@
test('Service Card mounted with vuex', async () => {
const wrapper = mount(ServiceCard, {
global: {
plugins: [store]
Owner

посмотри везде

посмотри везде
@ -0,0 +217,4 @@
expect(userWithoutServiceAfterDelete).toBeDefined()
expect(userWithoutServiceAfterDelete.id).toEqual(forDeleteUser.id)
}, 100)
Owner

Скорее всего можно без таймаута, позови меня посмотрим

Скорее всего можно без таймаута, позови меня посмотрим
SergeyV added 1 commit 2024-04-03 09:13:19 +00:00
SergeyV closed this pull request 2024-04-03 09:15:12 +00:00

Pull request closed

Sign in to join this conversation.
No Reviewers
No Label
No Milestone
No project
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: vbuglov/frontend-apps#2
No description provided.