move_users #2
Loading…
x
Reference in New Issue
Block a user
No description provided.
Delete Branch "move_users"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Updated users project
@ -0,0 +49,4 @@if (this.errorPassword) return}this.serviceOfUsers.editUserByService(params)},ревью для функции 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Такая запись путает, если if улетает на новую строку предланаю писать так:
если это сброс писать так:
@ -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) {Разнеси функцию
@ -37,0 +166,4 @@const users = this.store.getters['users/usersWithService']const updatedUsers = updatedUser(selectedUser, users)this.store.dispatch('users/updateUsersWithService', updatedUsers)// return updatedUserУдали комметарии которые не несут подсказок
@ -17,0 +25,4 @@async deleteUserFromService(userId) {const selectedService = await this.serviceOfServices.getSelectedService()if (!selectedService) {console.error('removeUserFromService: Service not found')logger.error
@ -17,0 +29,4 @@return {}}if (!userId) {console.error('removeUserFromService: User not selected')logger.error
@ -18,0 +38,4 @@async saveEditedData() {const selectedService = await this.serviceOfServices.getSelectedService()if (!selectedService) {console.error('saveEditedData: Service not found')logger.error
@ -18,0 +55,4 @@} else {this.serviceOfServices.saveService(selectedService)this.serviceOfUsers.saveUpdatedDataUser(selectedService.id)}разнеси функцию
@ -0,0 +33,4 @@"internet_uri": "localhost","description": "localhost","is_online": true},Много где используется вынеси в mock файл, у меня были примеры
const defaultServices = [
@ -0,0 +49,4 @@"internet_uri": "localhost","description": "localhost","is_online": true},Много где используется вынеси в mock файл, у меня были примеры
const defaultServices = [
@ -0,0 +71,4 @@const wrapper = mount(EditServiceCard, {global: {plugins: [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:
@ -0,0 +97,4 @@const wrapper = mount(EditServiceCard, {global: {plugins: [store]const wrapper = mount(App, {
global: {
mocks: {
$store
}
}
})
@ -0,0 +143,4 @@const wrapper = mount(EditServiceCard, {global: {plugins: [store]const wrapper = mount(App, {
global: {
mocks: {
$store
}
}
})
@ -0,0 +36,4 @@"internet_uri": "localhost","description": "localhost","is_online": true},Много где используется вынеси в mock файл, у меня были примеры
const defaultServices = [
...
посмотри везде
@ -0,0 +76,4 @@test('Service Card mounted with vuex', async () => {const wrapper = mount(ServiceCard, {global: {plugins: [store]посмотри везде
@ -0,0 +217,4 @@expect(userWithoutServiceAfterDelete).toBeDefined()expect(userWithoutServiceAfterDelete.id).toEqual(forDeleteUser.id)}, 100)Скорее всего можно без таймаута, позови меня посмотрим
Pull request closed