chore(break)!: remove unused methods

Author: chantouchsek

jest.config.ts

@@ -5,10 +5,8 @@ const config: InitialOptionsTsJest = {
   collectCoverageFrom: ['src/**/*.ts', 'src/**/*.js'],
   coverageThreshold: {
     global: {
-      lines: 100,
-      functions: 100,
-      branches: 92.72,
-      statements: 99.33,
+      branches: 93,
+      statements: 99,
     },
   },
   testEnvironment: 'node',

src/__tests__/base-proxy.test.ts

@@ -209,7 +209,7 @@ describe('BaseProxy', () => {
       return [200, {}]
     })
 
-    await proxy.create(form)
+    await proxy.store(form)
   })
 
   it('transforms the data to a FormData object if there is a File with post', async () => {
@@ -241,7 +241,7 @@ describe('BaseProxy', () => {
       return [200, {}]
     })
 
-    await proxy.putWithFile(form.id, form)
+    await proxy.put(form.id, form)
   })
 
   it('transforms the boolean values in FormData object to "1" or "0"', async () => {
@@ -306,7 +306,7 @@ describe('BaseProxy', () => {
   it('it should be able to put item without id parameter', async () => {
     const item = { first_name: 'Chantouch', last_name: 'Sek', id: 1 }
     mockAdapter.onPut('posts').reply(200, { data: item })
-    const { data } = await proxy.putWithoutId(item)
+    const { data } = await proxy.put(item)
     expect(data).toEqual(item)
   })
 
@@ -317,6 +317,13 @@ describe('BaseProxy', () => {
     expect(data).toEqual(item)
   })
 
+  it('it should be able to patch item without id', async () => {
+    const item = { first_name: 'Chantouch', last_name: 'Sek', id: 1 }
+    mockAdapter.onPatch('posts').reply(200, { data: item })
+    const { data } = await proxy.patch(item)
+    expect(data).toEqual(item)
+  })
+
   it('it should be able to update an item', async () => {
     const item = { first_name: 'Chantouch', last_name: 'Sek', id: 1 }
     mockAdapter.onPatch('posts/1').reply(200, { data: item })

src/__tests__/post-proxy.spec.ts

@@ -29,7 +29,6 @@ describe('PostProxy', () => {
       items: data,
       pagination: meta.pagination,
     }
-    console.warn('meta', item.pagination)
     expect(meta).toHaveProperty('pagination')
     expect(data.length).toEqual(1)
     expect(item.pagination.page).toEqual(1)

src/core/BaseProxy.ts

@@ -7,7 +7,12 @@ import type {
 } from 'axios'
 import type { Errors } from '..'
 import Validator from './Validator'
-import { hasFiles, objectToFormData, removeDoubleSlash } from '../util'
+import {
+  hasFiles,
+  objectToFormData,
+  removeDoubleSlash,
+  isObject,
+} from '../util'
 import qs, { IParseOptions } from 'qs'
 
 const validator = Validator
@@ -51,37 +56,46 @@ class BaseProxy {
     return this.submit<T>('get', id)
   }
 
+  post<T>(payload: any): Promise<T>
+  post<T>(payload: any, config?: AxiosRequestConfig): Promise<T>
   post<T>(payload: any, config?: AxiosRequestConfig) {
     return this.submit<T>('post', '', payload, config)
   }
 
+  store<T>(payload: any): Promise<T>
+  store<T>(payload: any, config?: AxiosRequestConfig): Promise<T>
   store<T>(payload: any, config?: AxiosRequestConfig) {
     return this.post<T>(payload, config)
   }
 
-  create<T>(payload: any, config?: AxiosRequestConfig) {
-    return this.store<T>(payload, config)
-  }
-
-  put<T>(id: string | number, payload: any) {
-    return this.submit<T>('put', `/${id}`, payload)
-  }
-
-  putWithoutId<T>(payload: any) {
-    return this.submit<T>('put', '', payload)
-  }
-
-  putWithFile<T>(
+  put<T>(payload: any): Promise<T>
+  put<T>(id: string | number, payload: any): Promise<T>
+  put<T>(payload: any, config?: AxiosRequestConfig): Promise<T>
+  put<T>(
     id: string | number,
     payload: any,
     config?: AxiosRequestConfig,
-  ) {
-    payload._method = 'put'
-    return this.submit<T>('post', `/${id}`, payload, config)
+  ): Promise<T>
+  put<T>(id: string | number, payload?: any, config?: AxiosRequestConfig) {
+    const parameter = id && !isObject(id) ? `/${id}` : ''
+    const requestType: Method = hasFiles(payload) ? 'post' : 'put'
+    if (hasFiles(payload)) {
+      Object.assign(payload, { _method: 'put' })
+    }
+    return this.submit<T>(requestType, parameter, payload, config)
   }
 
-  patch<T>(id: string | number, payload: any) {
-    return this.submit<T>('patch', `/${id}`, payload)
+  patch<T>(payload: any): Promise<T>
+  patch<T>(id: string | number, payload: any): Promise<T>
+  patch<T>(payload: any, config?: AxiosRequestConfig): Promise<T>
+  patch<T>(
+    id: string | number,
+    payload: any,
+    config?: AxiosRequestConfig,
+  ): Promise<T>
+  patch<T>(id: string | number, payload?: any, config?: AxiosRequestConfig) {
+    const parameter = id && !isObject(id) ? `/${id}` : ''
+    return this.submit<T>('patch', parameter, payload, config)
   }
 
   update<T>(id: string | number, payload: any) {